Approve commit access once per user #84
Comments
https://admin.fedoraproject.org/pkgdb/acl/pending/ has a button to approve/deny all pending requests already :) |
Isn't it too big hammer? |
Moreover, once you accept the ACLs, you can't deny them using this button |
On Thu, 2014-08-28 at 11:49 -0700, Vít Ondruch wrote:
This sentence confuses me :) |
Well, the pending acls page can be used just if somebody asks you for confirmation. But if you decide for whatever reason, that you want to deny already assigned rights, it is useless. |
On Thu, Aug 28, 2014 at 12:17:42PM -0700, Vít Ondruch wrote:
I agree but it sounds quite logical to me, if the ACLs are assigned, they are |
Yes, I don't dispute that :) The only thing I wanted to say is that it does not fit exactly my RFE |
I would prefer if this could be implemented as a bit of UI: something like an [Approve requested] button. |
The current behavior is more than [Approve All] it can also be [Deny All], you could use it to make everything Denied but the one you're interested in granted or vice-versa |
When somebody asks me for permissions of all Fedora/EPEL branches, I have to go through every branch column in Edit Commit Access page [1] and set it explicitly on approved. Is there chance to approve them in once?
It might be some additional column? May be it could be just some client side JS, which would update combo states in every branch column? When I have no right to approve right for some branch, the setting will fail just for that specific branch (actually this works already, so it should not be any issue).
[1] https://admin.fedoraproject.org/pkgdb/package/rubygem-rack/acl/commit/
The text was updated successfully, but these errors were encountered: