Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Customize bounce template for mailserver #508

Merged
merged 5 commits into from
Dec 1, 2024

Conversation

jfly
Copy link
Contributor

@jfly jfly commented Nov 9, 2024

The default template tells you to "please send mail to postmaster" which isn't very actionable (what's the postmaster?). This directs people to a place where they can actually get help.

Before

2024-11-09_11-48-13_pattern

After

2024-11-09_12-25-36_pattern

Alternatives considered

  • We could set up a mailing list for the infra team, and direct people to it
  • We could direct people to reach out to the infra team over matrix

@jfly jfly requested a review from Mic92 November 9, 2024 18:28
@jfly jfly force-pushed the custom-bounce-template branch from 9051d6d to 5051e2f Compare November 9, 2024 18:30
@jfly jfly force-pushed the custom-bounce-template branch 2 times, most recently from a692f98 to 5633478 Compare November 9, 2024 18:47
@jfly jfly requested a review from mweinelt November 9, 2024 18:47
@Mic92 Mic92 enabled auto-merge December 1, 2024 18:42
jfly and others added 5 commits December 1, 2024 20:17
The default template tells you to "please send mail to postmaster"
which isn't very actionable (what's the postmaster?). This directs
people to a place where they can actually get help.
this is the default already.
@Mic92 Mic92 force-pushed the custom-bounce-template branch from 85506f7 to c8f9f73 Compare December 1, 2024 19:39
@Mic92 Mic92 requested a review from a team as a code owner December 1, 2024 19:39
@Mic92 Mic92 merged commit 395ca8f into NixOS:master Dec 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants