Page MenuHomePhabricator

Apply reviewer changes from Herald via ModularTransactions
ClosedPublic

Authored by epriestley on Mar 20 2017, 3:47 PM.
Tags
None
Referenced Files
F15451087: D17511.id42120.diff
Fri, Mar 28, 8:03 PM
F15450197: D17511.id42136.diff
Fri, Mar 28, 2:29 PM
F15436240: D17511.id.diff
Tue, Mar 25, 12:48 PM
F15433697: D17511.diff
Tue, Mar 25, 12:18 AM
F15423524: D17511.id42136.diff
Sat, Mar 22, 3:49 PM
F15423493: D17511.id42120.diff
Sat, Mar 22, 3:41 PM
F15401119: D17511.id42136.diff
Mon, Mar 17, 3:50 PM
F15377098: D17511.id42136.diff
Mar 13 2025, 7:59 AM
Subscribers
None

Details

Summary

Ref T10967. This converts the reviewer update action in Herald from an older edge write to a newer ModularTransactions write.

The major value from this is that we get a double-write to the new reviewers table.

Test Plan
  • Wrote a Herald rule to add a reviewer and a blocking reviewer.
  • Saw them added properly to a revision with: no reviewers; both as blocking; A as blocking, B as nonblocking; A as nonblocking, B as blocking.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Mar 20 2017, 4:05 PM
This revision was automatically updated to reflect the committed changes.