pagure

#2580 Remove existing access when giving a project to a user who already has access
Merged by pingou. Opened by mprahl.
Unknown source master

Download 2580.patch

Addresses #2579

Looks fine to me

Honestly, first I would add user as owner and only after remove his other ACLS. e.g. in case something goes wrong, he will still have access.

But all my words might be irrelevant if we commit everything at once (which should be right way)..

In any case, :beers:

we commit everything at once

Looking at the code I believe we do that :)

rebased onto 10dcea45cdefdc79d94f2b2bc016bc34a08a1d93

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.