Page MenuHomePhabricator

Move all revision status transactions to modern values and mechanics
ClosedPublic

Authored by epriestley on Aug 12 2017, 12:03 AM.
Tags
None
Referenced Files
F18841187: D18419.id.diff
Tue, Oct 28, 7:33 AM
F18839742: D18419.diff
Mon, Oct 27, 6:37 PM
F18830305: D18419.id.diff
Sat, Oct 25, 5:59 AM
F18771853: D18419.diff
Wed, Oct 8, 10:21 PM
F18696635: D18419.id44254.diff
Sep 27 2025, 8:09 AM
F18642177: D18419.diff
Sep 18 2025, 10:49 PM
F18509007: D18419.id.diff
Sep 5 2025, 3:09 AM
F18501430: D18419.diff
Sep 4 2025, 9:45 PM
Subscribers
None

Details

Summary

Ref T2543. This updates and migrates the status change transactions:

  • All storage now records the modern modular transaction ("differential.revision.status"), not the obsolete non-modular transaction ("differential:status").
  • All storage now records the modern constants ("accepted"), not the obsolete numeric values ("2").
Test Plan
  • Selected all the relevant rows before/after migration, data looked sane.
  • Browsed around, reviewed timelines, no changes after migration.
  • Changed revision states, saw appropriate new transactions in the database and timeline rendering.
  • Grepped for differential:status.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable