HomePhabricator

Remove defunct "metamta.herald.show-hints" Config option

Description

Remove defunct "metamta.herald.show-hints" Config option

Summary:
Ref T13216. See PHI985. This config option once controlled adding a Herald transcript link to email. However, this was never implemented in a generic way and was removed from revisions in D8459 and from commits in D10705. No one has noticed or asked for this option for several years, so this is probably a good opportunity to simplify the software and reduce the total amount of configuration.

If we did want to pursue this in the future, I'd generally prefer to make it part of the mail detail page (/mail/detail/12345/) anyway.

Test Plan: Grepped for metamta.herald.show-hints and addHeraldSection(), got no hits for either.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13216

Differential Revision: https://secure.phabricator.com/D19833

Details

Provenance
epriestleyAuthored on Nov 24 2018, 2:30 PM
epriestleyPushed on Nov 26 2018, 6:14 PM
Reviewer
amckinley
Differential Revision
D19833: Remove defunct "metamta.herald.show-hints" Config option
Parents
rP9473f60a362f: Allow "Abandoned" revisions to be commandeered
Branches
Unknown
Tags
Unknown
Tasks
T13216: 2018 Week 45-47 Bonus Content
Build Status
Buildable 21224
Build 28868: Run Core Tests