Page MenuHomePhabricator

Fix two transactional issues around revision status
ClosedPublic

Authored by epriestley on Mar 12 2014, 8:44 PM.
Tags
None
Referenced Files
F14056950: D8506.diff
Sat, Nov 16, 11:33 PM
F13999600: D8506.diff
Thu, Oct 24, 3:52 PM
Unknown Object (File)
Oct 9 2024, 3:37 PM
Unknown Object (File)
Oct 8 2024, 10:44 AM
Unknown Object (File)
Sep 11 2024, 12:31 AM
Unknown Object (File)
Sep 5 2024, 3:06 AM
Unknown Object (File)
Aug 29 2024, 6:14 AM
Unknown Object (File)
Aug 25 2024, 11:08 PM
Subscribers
Tokens
"Mountain of Wealth" token, awarded by dctrwatson.

Details

Summary

Two issues:

  • Herald is currently overwriting accepts and rejects with "blocking reviewer". Just stop it from doing that.
  • When you update an accepted revision, we put it back in "needs review", then return it to "accepted", generating an extra transaction. Instead, don't.
Test Plan
  • Updated a revision with an accepting, herald-based blocking project reviewer. Reviewer was still accepting.
  • Updated an accepted revision, didn't get an extra transaction.

Diff Detail

Repository
rP Phabricator
Branch
projblock
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

epriestley retitled this revision from to Fix two transactional issues around revision status.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 12 2014, 9:49 PM
epriestley updated this revision to Diff 20187.

Closed by commit rP3b861ab7411d (authored by @epriestley).