Page MenuHomePhabricator

Add a setting to disable all notification email
ClosedPublic

Authored by epriestley on Aug 12 2014, 5:27 PM.
Tags
None
Referenced Files
F14061903: D10237.diff
Mon, Nov 18, 9:16 AM
F14046084: D10237.diff
Wed, Nov 13, 6:03 PM
F14034851: D10237.diff
Sun, Nov 10, 2:30 AM
F14032367: D10237.id24630.diff
Sat, Nov 9, 2:12 PM
F14020796: D10237.diff
Wed, Nov 6, 2:40 AM
F13959135: D10237.id.diff
Oct 14 2024, 5:59 PM
Unknown Object (File)
Oct 7 2024, 11:23 PM
Unknown Object (File)
Oct 7 2024, 11:23 PM

Details

Summary

Ref T5861. Adds an option to opt out of all notification email. We'll still send you password resets, email verifications, etc.

Test Plan

Screen_Shot_2014-08-12_at_10.25.05_AM.png (652×1 px, 151 KB)

  • Added unit tests.
  • With preference set to different things, tried to send myself mail. Mail respected preferences.
  • Sent password reset email, which got through the preference.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Add a setting to disable all notification email.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 12 2014, 5:34 PM
epriestley updated this revision to Diff 24641.

Closed by commit rPd011f8fdc6c4 (authored by @epriestley).