Add Maniphest Task email creator to CCs
ClosedPublic
Actions

Authored by epriestley on Sep 9 2015, 8:11 PM.

Details

Reviewers

chad

Maniphest Tasks

T9369: Author of email not added to maniphest subscribers

Commits

Summary

Fixes T9369.

Test Plan

Sent a mail with Mail.app to bugs@local.phacility.com.
Used "View Raw Mail", copy-pasted it into mail.txt on disk.
Ran cat mail.txt | ./scripts/mail/manage_mail.php --process-duplicates.
Saw task get created and me get added as CC.
Changed "To" to include another user, ran command again, saw task get created and other user get added as CC.

Repository

rP Phabricator

Branch

taskmail1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 7949
Build 8928: [Placeholder Plan] Wait for 30 Seconds
Build 8927: arc lint + arc unit

chad retitled this revision from to Add Maniphest Task email creator to CCs.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

This won't actually apply the transaction and $sender may not have a real PHID, let me poke this and kick it back to you.

I assumed this was automatic in ManiphestTask, but in the edit page we set it by hand.

epriestley edited the test plan for this revision. (Show Details)

epriestley edited edge metadata.

For Maniphest, add author as subscriber.
Fix "subscribe all To/Cc" which got derped up a while ago (this is presumably rarely used).
Some minor cleanup.