Page MenuHomePhabricator

MetaMTA - update documentation and make config a tad easier
ClosedPublic

Authored by btrahan on Feb 10 2015, 12:42 AM.
Tags
None
Referenced Files
F14064336: D11725.diff
Mon, Nov 18, 11:00 PM
F14036389: D11725.diff
Sun, Nov 10, 10:04 AM
F14035529: D11725.diff
Sun, Nov 10, 6:20 AM
F13997865: D11725.diff
Thu, Oct 24, 6:34 AM
F13988179: D11725.id28344.diff
Mon, Oct 21, 1:42 PM
F13983672: D11725.id28238.diff
Oct 20 2024, 8:01 AM
F13966207: D11725.diff
Oct 16 2024, 6:56 AM
F13962904: D11725.id28344.diff
Oct 15 2024, 12:53 PM
Subscribers

Details

Summary

Fixes T7088. Mainly this updates the documentation but I also snuck in tweaking how the domain reply handler is built. This does two main things -- makes the behavior consistent as some applications who didn't override this behavior would send out emails with reply tos AND makes it easier for us to deprecate the custom domain thing on a per application basis, which is just silly. On that note, the main documentation doesn't get into how this can be overridden, though I left in that mini blurb on the config setting itself. We could deprecate this harder and LOCK things if you want as well.

Test Plan

read docs, looked good. reasoned through re-factor

Diff Detail

Repository
rP Phabricator
Branch
T7088
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4414
Build 4427: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan retitled this revision from to MetaMTA - update documentation and make config a tad easier.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/docs/user/configuration/configuring_inbound_email.diviner
61–66

Maaaaybe just omit this since we're probably going to forget to update it until the next major change requires us to revise this doc, and I'd guess that use of these features is need-driven ("I want to create tasks via email, let me figure out how") rather than opportunity-driven ("I can create tasks via email, so I'll do that"), although maybe not. I dunno.

Alternatively, we could add an option to Applications to let you filter by applications with email support to make sure this information is always available and accurate.

This revision is now accepted and ready to land.Feb 12 2015, 6:12 PM
btrahan edited edge metadata.

refine documentation by dropping the list
add support for searching for app email support in /applications/

This revision was automatically updated to reflect the committed changes.