Open Bug 1471341 Opened 6 years ago Updated 5 years ago

Warn user when viewing a revision that they have admin rights and their account is not linked to BMO

Categories

(Conduit :: Phabricator, enhancement, P3)

Product:
Conduit
Component:
Phabricator
Type:
enhancement

Tracking

(Not tracked)

Status:
ASSIGNED

People

(Reporter: dkl, Assigned: dkl)

Details

(Keywords: conduit-triaged) 

This only matters really for revisions with bug id links, but we want to discourage revision changes from a user account that has admin rights and/or does not have a link to a valid BMO account. This causes issues with the Phabbugz daemon in BMO when it see changes made by users who do not have related BMO accounts.

dkl
this shouldn't be a warning - we should outright prevent admin users from conducting reviews.
(In reply to Byron Jones ‹:glob› from comment #1)
> this shouldn't be a warning - we should outright prevent admin users from
> conducting reviews.

This seems a little overkill to me, there are a limited number of admins that should just not do it IMO. Spending time modifying Phabricator to make it impossible sounds fragile and not really worth it.
Keywords: conduit-triaged
Whiteboard: [phabricator-backlog]
Keywords: conduit-backlog
Whiteboard: [phabricator-backlog]
Keywords: conduit-backlog
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.