Page MenuHomePhabricator
Differential D13705

Modularize "add projects" and "remove projects" Herald actions
ClosedPublic
Actions

Authored by epriestley on Jul 24 2015, 7:26 PM.
Tags
None
Referenced Files
F15531358: D13705.diff
Wed, Apr 23, 10:54 AM
F15519533: D13705.id33280.diff
Sat, Apr 19, 11:36 PM
F15510666: D13705.id33280.diff
Wed, Apr 16, 8:29 PM
F15510665: D13705.id.diff
Wed, Apr 16, 8:29 PM
F15510664: D13705.diff
Wed, Apr 16, 8:29 PM
F15507068: D13705.id33280.diff
Tue, Apr 15, 4:06 PM
F15504984: D13705.id33123.diff
Mon, Apr 14, 11:07 PM
F15503895: D13705.id33129.diff
Mon, Apr 14, 2:14 PM
View All Files
Subscribers
epriestley
joshuaspence
Tokens
"Orange Medal" token, awarded by joshuaspence.

Details

Reviewers
btrahan
Maniphest Tasks
T8726: Modularize Herald fields and actions
Commits
Restricted Diffusion Commit
rP37829926702b: Modularize "add projects" and "remove projects" Herald actions
Summary

Ref T8726. Convert these to be modular.

Test Plan
  • Created rules using these actions.
  • Upgraded them.
  • Verified they still work.

Screen Shot 2015-07-24 at 12.24.07 PM.png (200×495 px, 33 KB)

Diff Detail

Repository
rP Phabricator
Branch
heraldx6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7383
Build 7806: [Placeholder Plan] Wait for 30 Seconds
Build 7805: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 33123.Jul 24 2015, 7:26 PM
epriestley retitled this revision from to Modularize "add projects" and "remove projects" Herald actions.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T8726: Modularize Herald fields and actions.
Herald added a subscriber: epriestley. · View Herald TranscriptJul 24 2015, 7:26 PM
btrahan accepted this revision.Jul 25 2015, 12:01 AM
btrahan edited edge metadata.
btrahan added inline comments.
src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php
1244

duplicate s

This revision is now accepted and ready to land.Jul 25 2015, 12:01 AM
epriestley updated this revision to Diff 33129.Jul 25 2015, 12:03 AM
epriestley edited edge metadata.
  • Fix duplicate "ss".
joshuaspence awarded a token.Jul 25 2015, 1:23 AM
joshuaspence rescinded a token.
joshuaspence awarded a token.
joshuaspence added a subscriber: joshuaspence.
Closed by commit rP37829926702b: Modularize "add projects" and "remove projects" Herald actions (authored by epriestley, committed by epriestley). · Explain WhyAug 3 2015, 9:52 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
11 lines
src/
6 lines
applications/
herald/
adapter/
58 lines
project/
herald/
28 lines
180 lines
28 lines
infrastructure/
internationalization/
translation/
20 lines

Diff 33123

View Options

resources/sql/autopatches/20150724.herald.3.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
View Options

src/applications/project/herald/PhabricatorProjectAddHeraldAction.php

Loading...
View Options

src/applications/project/herald/PhabricatorProjectHeraldAction.php

Loading...
View Options

src/applications/project/herald/PhabricatorProjectRemoveHeraldAction.php

Loading...
View Options

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...
Phabricator · Built by Phacility