pagure

#2392 don't show '+' for groups when USER_MNGT off
Merged by pingou. Opened by vivekanand1101.
vivekanand1101/pagure grp  into  master

Download 2392.patch

Signed-off-by: Vivek Anand vivekanand1101@gmail.com

I couldn't run the test suite because it gets stuck at one of the tests :/

Do you have python-eventlet installed?

Do you have python-eventlet installed?

yes

Do you know which test is getting stuck?

api_pull_request_merge method

Running the tests :)

Shouldn't this be ENABLE_GROUP_MNGT ?

Actually, we should add tests for this, ensuring the button is indeed not showing up with group management is disabled.

Actually, we should add tests for this, ensuring the button is indeed not showing up with group management is disabled.

ok
Edited by pingou

rebased

rebased

rebased

@vivekanand1101 do you want to work on the tests or shall I ?

rebased

:thumbsup:

rebased

Pull-Request has been merged by pingou
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.