Page MenuHomePhabricator

Implicitly subscribe task author when they create a task
ClosedPublic

Authored by epriestley on Dec 7 2015, 7:11 PM.
Tags
None
Referenced Files
F13048520: D14694.id35537.diff
Thu, Apr 18, 8:37 PM
Unknown Object (File)
Wed, Apr 17, 9:41 PM
Unknown Object (File)
Fri, Mar 29, 11:16 AM
Unknown Object (File)
Mar 6 2024, 12:51 PM
Unknown Object (File)
Mar 5 2024, 7:09 PM
Unknown Object (File)
Mar 5 2024, 7:09 PM
Unknown Object (File)
Mar 5 2024, 7:05 PM
Unknown Object (File)
Feb 23 2024, 9:40 AM
Subscribers
None

Details

Summary

Ref T9908. This is a behavioral change:

  • Old behavior: "Subscribers" field is default-populated with author.
  • New behavior: this transaction is just created no matter what.

The new behavior is much easier to make work with form defaults, hidden fields, etc. For example, on the "Create Bug" form, I've hidden "Subscribers", but I still want the author to be subscribed.

And if a user sets the default value of "Subscribers:" to "Alice, Bob", they almost certainly mean "Alice, Bob, and the task author".

And I ended up deleting myself by accident way more often than I deleted myself on purpose -- especially with "Create Similar task", I'd sometimes delete all the CCs and delete myself by accident and then have to put myself back.

Finally, technically speaking, restoring the old behavior is kind of hard/messy and this is much easier.

Test Plan
  • Created a task.
  • Was automatically added as a subscriber.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Implicitly subscribe task author when they create a task.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 7 2015, 7:12 PM
This revision was automatically updated to reflect the committed changes.