Page MenuHomePhabricator

Maniphest - always cc the user, including when creating from template
ClosedPublic

Authored by btrahan on Sep 3 2014, 7:58 PM.
Tags
None
Referenced Files
F13101802: D10408.diff
Sat, Apr 27, 1:30 AM
F13101145: D10408.id.diff
Fri, Apr 26, 10:19 PM
Unknown Object (File)
Wed, Apr 24, 11:48 PM
Unknown Object (File)
Thu, Apr 4, 4:09 PM
Unknown Object (File)
Mar 26 2024, 10:16 AM
Unknown Object (File)
Mar 18 2024, 2:11 AM
Unknown Object (File)
Feb 2 2024, 9:16 AM
Unknown Object (File)
Jan 17 2024, 10:18 PM
Subscribers

Details

Summary

Ref T6031. I figure its totally cool to include the user creating the task as a subscriber, even if from the template case, so just do that there too. Code is written such that if the user wasn't already in the subscriber case they end up being the last person in the tokenizer. Theoretically this should make any users who didn't want to be automagically subscribed via the create from template case to remove themselves.

Test Plan

made a template from a task that didn't have me as a subscriber initially and observed i was a subscriber.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan retitled this revision from to Maniphest - always cc the user, including when creating from template.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Sep 3 2014, 8:03 PM
btrahan edited edge metadata.
btrahan updated this revision to Diff 25043.

Closed by commit rPb60d0c338a20 (authored by @btrahan).