Page MenuHomePhabricator

Apply "accept" and "resign" actions with transactions
ClosedPublic

Authored by epriestley on Aug 2 2014, 7:00 PM.
Tags
None
Referenced Files
F19019038: D10125.id24352.diff
Nov 23 2025, 1:16 PM
F18877835: D10125.diff
Nov 6 2025, 12:19 PM
F18846663: D10125.diff
Oct 29 2025, 8:25 PM
F18825626: D10125.id.diff
Oct 23 2025, 11:06 PM
F18774963: D10125.id24362.diff
Oct 10 2025, 5:29 PM
F18772498: D10125.id24352.diff
Oct 9 2025, 2:52 AM
F18756994: D10125.id24352.diff
Oct 5 2025, 2:56 PM
F18651982: D10125.diff
Sep 21 2025, 6:11 AM
Subscribers

Details

Summary

Ref T4896. Applies these actions using new transaction stuff.

Test Plan
  • Accepted and raised concern with my own commit, verifying the special project/package behavior.
  • Accepted and raised concern with another author's commit, verifying the authority-over-packages/projects behavior.
  • Accepted a commit I was not affiliated wiht, verifying the "join as an auditor" behavior.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Apply "accept" and "resign" actions with transactions.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, joshuaspence.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 2 2014, 7:14 PM
epriestley updated this revision to Diff 24362.

Closed by commit rPa297450aa97b (authored by @epriestley).