Page MenuHomePhabricator

Modularize mail tags
ClosedPublic

Authored by epriestley on Aug 12 2014, 6:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 3, 9:55 PM
Unknown Object (File)
Fri, Nov 29, 10:34 PM
Unknown Object (File)
Fri, Nov 29, 10:34 PM
Unknown Object (File)
Fri, Nov 29, 10:34 PM
Unknown Object (File)
Thu, Nov 28, 2:03 AM
Unknown Object (File)
Tue, Nov 26, 7:53 PM
Unknown Object (File)
Sun, Nov 24, 8:46 AM
Unknown Object (File)
Nov 3 2024, 1:42 AM
Subscribers

Details

Reviewers
btrahan
chad
Maniphest Tasks
T5861: Improve the power of email settings
Commits
Restricted Diffusion Commit
rPf6f9d78f3a07: Modularize mail tags
Summary

Ref T5861. Currently, mail tags are hard-coded; move them into applications. Each Editor defines its own tags.

This has zero impact on the UI or behavior.

Test Plan
  • Checked/unchecked some options, saved form.
  • Swapped back to master and saw exactly the same values.

Diff Detail

Repository
rP Phabricator
Branch
mail3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 2177
Build 2181: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Modularize mail tags.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.

This has zero impact on the UI or behavior.

Actually, it used to say "Differential" and now it says "Differential Revisions" so I guess that's not 100% true.

(I qualified the object type to accommodate applications which have several editable objects.)

btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 12 2014, 7:07 PM
epriestley updated this revision to Diff 24642.

Closed by commit rPf6f9d78f3a07 (authored by @epriestley).