Page MenuHomePhabricator

Modularize Herald "flag" action, plus update transcripts
ClosedPublic

Authored by epriestley on Jul 18 2015, 1:07 PM.
Tags
None
Referenced Files
F15187425: D13649.id32996.diff
Fri, Feb 21, 8:50 AM
F15187418: D13649.id33277.diff
Fri, Feb 21, 8:47 AM
F15187409: D13649.id33277.diff
Fri, Feb 21, 8:46 AM
F15187371: D13649.diff
Fri, Feb 21, 8:22 AM
F15187369: D13649.id32996.diff
Fri, Feb 21, 8:22 AM
F15187366: D13649.id.diff
Fri, Feb 21, 8:22 AM
F15187360: D13649.diff
Fri, Feb 21, 8:21 AM
F15187358: D13649.id.diff
Fri, Feb 21, 8:21 AM

Details

Summary

Ref T8726. This modularizes "Mark with flag", plus rebuilds transcripts in a more modern/flexible way. The big transcript stuff is:

  • Transcripts are now translatable.
  • Transcripts can now show multiple outputs from a single action. For example, an action like "add A, B, C to subscribers" can now say "added A; B is invalid; C was already subscribed".
Test Plan

Screen Shot 2015-07-18 at 5.53.53 AM.png (954×2 px, 165 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Modularize Herald "flag" action, plus update transcripts.
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.Jul 20 2015, 6:20 PM
This revision was automatically updated to reflect the committed changes.