Page MenuHomePhabricator

Don't raise the "Subscribers Won't Be Notified" draft warning if you aren't adding any non-you subscribers
ClosedPublic

Authored by epriestley on Jul 9 2020, 9:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 1, 3:32 AM
Unknown Object (File)
Feb 13 2024, 9:00 PM
Unknown Object (File)
Jan 6 2024, 8:35 PM
Unknown Object (File)
Jan 3 2024, 6:31 PM
Unknown Object (File)
Dec 30 2023, 4:32 PM
Unknown Object (File)
Dec 26 2023, 7:17 PM
Unknown Object (File)
Dec 26 2023, 6:47 AM
Unknown Object (File)
Dec 23 2023, 5:11 AM
Subscribers
None

Details

Summary

Currently, adding subscribers to a draft revision raises a warning that they won't get an email/notification.

This warning has some false positives:

  • it triggers on any subscriber change, including removing subscribers; and
  • it triggers if you're only adding yourself as a subscriber.

Narrow the scope of the warning so it is raised only if you're adding a subscriber other than yourself.

Test Plan
  • Added a non-self subscriber, got the warning as before.
  • Added self as a subscriber, no warning (previously: warning).
  • Removed a subscriber, no warning (previously: warning).

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.Jul 9 2020, 9:20 PM
This revision was automatically updated to reflect the committed changes.