Page MenuHomePhabricator
Differential D7355

Fix "Manage herald rules" link by removing it
ClosedPublic
Actions

Authored by epriestley on Oct 18 2013, 11:54 PM.
Tags
None
Referenced Files
F13218160: D7355.diff
Sat, May 18, 10:30 AM
F13207703: D7355.id16606.diff
Thu, May 16, 2:38 AM
F13207702: D7355.id16562.diff
Thu, May 16, 2:38 AM
F13207659: D7355.id.diff
Thu, May 16, 2:09 AM
F13207639: D7355.diff
Thu, May 16, 1:58 AM
Unknown Object (File)
Sun, Apr 28, 10:06 AM
Unknown Object (File)
Sat, Apr 20, 6:04 PM
Unknown Object (File)
Apr 11 2024, 2:52 AM
View All Files
Subscribers
aran

Details

Reviewers
zeeg
btrahan
Maniphest Tasks
T4001: "MANAGE HERALD RULES" link is broken
Commits
Restricted Diffusion Commit
rP00bf47f973c8: Fix "Manage herald rules" link by removing it
Summary

Fixes T4001. I broke this some time ago and no one has complained. I don't think it gets much use, and we haven't added it for the newer apps. Just get rid of it rather than adapt the URIs for ApplicationSearch.

Test Plan

Unit tests, sent myself some email.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Revision Contents
Changeset List

PathSize
conf/
4 lines
src/
applications/
config/
option/
6 lines
differential/
mail/
3 lines
metamta/
view/
6 lines
__tests__/
8 lines
repository/
worker/
3 lines

Diff 16606

View Options

conf/default.conf.php

Loading...
View Options

src/applications/config/option/PhabricatorMetaMTAConfigOptions.php

Loading...
View Options

src/applications/differential/mail/DifferentialMail.php

Loading...
View Options

src/applications/metamta/view/PhabricatorMetaMTAMailBody.php

Loading...
View Options

src/applications/metamta/view/__tests__/PhabricatorMetaMTAMailBodyTestCase.php

Loading...
View Options

src/applications/repository/worker/PhabricatorRepositoryCommitHeraldWorker.php

Loading...
Phabricator · Built by Phacility