Page MenuHomePhabricator

Allow unsubscription from projects
ClosedPublic

Authored by epriestley on Feb 10 2014, 11:25 PM.
Tags
None
Referenced Files
F15477074: D8189.id18540.diff
Mon, Apr 7, 12:08 PM
F15471888: D8189.id.diff
Sat, Apr 5, 9:01 AM
F15469230: D8189.diff
Fri, Apr 4, 10:42 AM
F15428067: D8189.id18540.diff
Sun, Mar 23, 6:28 PM
F15427983: D8189.diff
Sun, Mar 23, 5:32 PM
F15417963: D8189.id18529.diff
Thu, Mar 20, 7:45 PM
F15398155: D8189.id.diff
Sun, Mar 16, 11:31 PM
F15368960: D8189.diff
Mar 12 2025, 4:45 AM
Subscribers

Details

Summary

Fixes T4379. Several changes:

  • Migrate all project members into subscribers.
  • When members are added or removed, subscribe or unsubscribe them.
  • Show sub/unsub in the UI.
  • Determine mailable membership of projects by querying subscribers.
Test Plan
  • As duck, joined a project.
  • Added the project as a reviewer to a revision.
  • Commented on the revision.
  • Observed duck receive mail.
  • Unsubscribed as duck.
  • Observed no mail.
  • Resubscribed as duck.
  • Mail again.
  • Joined/left project, checked sub/unsub status.
  • Ran migration, looked at database.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline