Page MenuHomePhabricator

Don't subscribe bots implicitly when they act on objects, or when they are mentioned
ClosedPublic

Authored by epriestley on Mar 13 2019, 6:48 PM.
Tags
None
Referenced Files
F14103629: D20284.id48420.diff
Tue, Nov 26, 10:20 PM
F14101023: D20284.id.diff
Tue, Nov 26, 3:44 PM
Unknown Object (File)
Mon, Nov 25, 8:23 PM
Unknown Object (File)
Sat, Nov 23, 7:26 AM
Unknown Object (File)
Thu, Oct 31, 5:53 AM
Unknown Object (File)
Oct 16 2024, 4:48 PM
Unknown Object (File)
Oct 6 2024, 2:09 AM
Unknown Object (File)
Oct 2 2024, 6:35 AM
Subscribers
None

Details

Summary

See PHI1098. When users comment on objects, they are automatically subscribed. And when @alice mentions @bailey on a task, that usually subscribes @bailey.

These rules make less sense if the user is a bot. There's generally no reason for a bot to automatically subscribe to objects it acts on (it's not going to read email and follow up later), and it can always subscribe itself pretty easily if it wants (since everything is *.edit now and supports subscribe transactions).

Also, don't subscribe bots when they're mentioned for similar reasons. If users really want to subscribe bots, they can do so explicitly.

These rules seem slightly like "bad implicit magic" since it's not immediately obvious why @abc subscribes that user but @xyz may not, but some of these rules are fairly complicated already (e.g., @xyz doesn't subscribe them if they unsubscribed or are implicitly subscribed) and this feels like it gets the right/desired result almost-always.

Test Plan

On a fresh task:

  • Mentioned a bot in a comment with @bot.
    • Before patch: bot got CC'd.
    • After patch: no CC.
  • Called maniphest.edit via the API to add a comment as a bot.
    • Before patch: bot got CC'd.
    • After patch: no CC.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable