fedora-hubs

#493 get basic allgroups page working
Merged by abompard. Opened by ryanlerch.
ryanlerch/fedora-hubs all-groups  into  develop

Download 493.patch

Updates the allgroups page.

This updates the page to kinda partially use react. We will probably want to update this in the future to a full react-style component, with a proper backend APO point so we can enable filtering on the list ajax-style, but for MVP, i think a simple static list of available teams hubs is a go!

I recommend removing this commented code.

Looks fine to me, though I recommend tests for the new Python code. Also, I don't really know React so my review isn't worth much on that code :)

whoops! yeah i need to uncomment that block, and remove the line below. The line below does the same thing, just isused to provide better debugging info (without bundling stuff).

Thanks for the review @bowlofeggs !

I have updated the commit to test the new python code i have added.

as well as fixing up the commented block in the javascrip that you pointed out.
Edited by ryanlerch

rebased onto 07ec473ffcd587927e84cd60ced5707e64ef5979

LGTM!

Pull-Request has been merged by abompard
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.