Page MenuHomePhabricator

Propagate "unexpandable" PHIDs to feed notification recipient expansion
ClosedPublic

Authored by epriestley on Mar 21 2018, 6:15 PM.
Tags
None
Referenced Files
F13087891: D19244.diff
Thu, Apr 25, 1:10 AM
Unknown Object (File)
Wed, Apr 17, 2:45 PM
Unknown Object (File)
Fri, Apr 5, 3:23 PM
Unknown Object (File)
Wed, Apr 3, 6:20 PM
Unknown Object (File)
Sun, Mar 31, 11:25 AM
Unknown Object (File)
Sun, Mar 31, 11:24 AM
Unknown Object (File)
Sun, Mar 31, 5:38 AM
Unknown Object (File)
Sun, Mar 31, 5:38 AM
Subscribers
None

Details

Summary

See PHI466. Ref T13108. Somewhat recently, new rules were added so that "Resigning" from a revision takes you off the default recipient list, even if you're still a member of a project or package that is still a reviewer or subscriber.

However, these rules don't currently apply to the similar expansion which occurs in notifications. If you resign from a revision you may still get some notifications (just not email) if a package or project you're a member of is a reviewer or subscriber.

(Possibly these should eventually share more code, but just get things working for now.)

Test Plan
  • Created a revision as A.
  • Added B as a reviewer.
  • Added a package B is an owner for as a reviewer.
  • As B, resigned. (Make sure B is also not an explicit subscriber.)
  • Commented on the revision as A.
    • Before: B is included in the expanded notification recipient list.
    • After: B is no longer included in the expanded notification recipient list.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 21 2018, 6:56 PM
This revision was automatically updated to reflect the committed changes.