Page MenuHomePhabricator

Allow users to re-accept or re-reject a revision if they have authority over package/project reviewers not yet in the target state
ClosedPublic

Authored by epriestley on Jan 18 2017, 9:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 3, 4:44 PM
Unknown Object (File)
Wed, Jul 1, 5:53 AM
Unknown Object (File)
Sun, Jun 28, 3:58 PM
Unknown Object (File)
Thu, Jun 25, 12:19 PM
Unknown Object (File)
Sun, Jun 21, 9:12 PM
Unknown Object (File)
Sat, Jun 20, 9:12 PM
Unknown Object (File)
Apr 22 2026, 7:19 PM
Unknown Object (File)
Apr 20 2026, 2:36 AM
Subscribers
None

Details

Summary

To set this up:

  • alice accepts a revision.
  • Something adds a package or project she has authority over as a reviewer.
  • Because alice has already accepted, she can not re-accept, but she should be able to (in order to accept on behalf of the new project or package).
Test Plan
  • Created a revision.
  • Accepted as user "dog".
  • Added "dog project".
  • Re-accepted.
  • Could not three-accept.
  • Removed "dog project.
  • Rejected.
  • Added "dog project".
  • Re-rejected.
  • Could not three-reject.

Diff Detail

Repository
rP Phabricator
Branch
fullaccept1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15319
Build 20167: Run Core Tests
Build 20166: arc lint + arc unit