Page MenuHomePhabricator

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

Authored by epriestley on Nov 24 2018, 2:34 PM.
Tags
None
Referenced Files
F18882161: D19833.id47375.diff
Nov 6 2025, 10:47 PM
F18878120: D19833.diff
Nov 6 2025, 3:12 PM
F18874468: D19833.id.diff
Nov 5 2025, 2:37 PM
F18774008: D19833.id47358.diff
Oct 9 2025, 2:20 PM
F18731296: D19833.id.diff
Sep 30 2025, 2:29 PM
F18714326: D19833.diff
Sep 29 2025, 7:32 AM
F18568981: D19833.id47375.diff
Sep 10 2025, 12:33 AM
F18568979: D19833.id47358.diff
Sep 10 2025, 12:33 AM
Subscribers
None

Details

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.

Diff Detail

Repository
rP Phabricator
Branch
hhints1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21207
Build 28841: Run Core Tests
Build 28840: arc lint + arc unit