Page MenuHomePhabricator

When users resign from revisions, stop expanding projects/packages to include them
ClosedPublic

Authored by epriestley on Feb 7 2018, 4:08 PM.
Tags
None
Referenced Files
F19301251: D19021.id45590.diff
Tue, Dec 23, 3:07 PM
F19024783: D19021.diff
Nov 24 2025, 4:24 AM
F19004862: D19021.diff
Nov 21 2025, 12:44 PM
F18934860: D19021.diff
Nov 10 2025, 5:05 PM
F18848948: D19021.diff
Oct 30 2025, 7:01 AM
F18821852: D19021.diff
Oct 23 2025, 2:14 AM
F18758416: D19021.id45590.diff
Oct 5 2025, 10:17 PM
F18745426: D19021.id45590.diff
Oct 3 2025, 6:44 AM
Subscribers
None

Details

Summary

Depends on D19019. Ref T13053. Fixes T12689. See PHI178.

Currently, if @alice resigns from a revision but #alice-fan-club is still a subscriber or reviewer, she'll continue to get mail. This is undesirable.

When users are associated with an object but have explicitly disengaged in an individal role (currently, only resign in audit/differential) mark them "unexpandable", so that they can no longer be included through implicit membership in a group (a project or package).

@alice can still get mail if she's a explicit recipient: as an author, owner, or if she adds herself back as a subscriber.

Test Plan
  • Added @ducker and #users-named-ducker as reviewers. Ducker got mail.
  • Resigned as ducker, stopped getting future mail.
  • Subscribed explicitly, got mail again.
  • (Plus some var_dump() sanity checking in the internals.)

Diff Detail

Repository
rP Phabricator
Branch
mailer18
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19384
Build 26218: Run Core Tests
Build 26217: arc lint + arc unit