Page MenuHomePhabricator

Modularize all Diffusion commit Herald fields
ClosedPublic

Authored by epriestley on Jul 6 2015, 9:15 PM.
Tags
None
Referenced Files
F15392679: D13567.id.diff
Sat, Mar 15, 4:52 PM
F15392342: D13567.id32827.diff
Sat, Mar 15, 2:39 PM
F15390707: D13567.id32788.diff
Sat, Mar 15, 6:28 AM
F15336100: D13567.diff
Sat, Mar 8, 6:14 PM
F15332971: D13567.id32827.diff
Fri, Mar 7, 10:46 PM
F15284535: D13567.diff
Tue, Mar 4, 9:56 AM
Unknown Object (File)
Thu, Feb 27, 5:04 AM
Unknown Object (File)
Feb 11 2025, 9:37 PM
Subscribers

Details

Summary

Ref T8726.

Test Plan

Created a giant rule with every commit field:

Screen Shot 2015-07-06 at 1.57.11 PM.png (1×1 px, 271 KB)

Ran the upgrade, got the same rule with new fields:

Screen Shot 2015-07-06 at 2.08.34 PM.png (1×1 px, 268 KB)

Used "Test Console" to run transcripts, saw all the fields populate correctly.

Diff Detail

Repository
rP Phabricator
Branch
hmodular10
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/applications/conpherence/query/ConpherenceThreadSearchEngine.php:21PHL1Unknown Symbol
Unit
Tests Passed
Build Status
Buildable 7164
Build 7373: [Placeholder Plan] Wait for 30 Seconds
Build 7372: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Modularize all Diffusion commit Herald fields.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.

I think 20150706.herald.1.sql is the MVP of this diff, possibly HeraldSubscribersField.php since it got to be the "parent of" (copied to) basically every other file.

This revision is now accepted and ready to land.Jul 7 2015, 6:36 PM
This revision was automatically updated to reflect the committed changes.