Page MenuHomePhabricator

Allow Maniphest to process multiple commands via email
ClosedPublic

Authored by epriestley on Apr 1 2015, 12:17 AM.
Tags
None
Referenced Files
F15306833: D12234.id29393.diff
Thu, Mar 6, 4:26 AM
Unknown Object (File)
Thu, Feb 27, 8:30 AM
Unknown Object (File)
Fri, Feb 21, 11:52 AM
Unknown Object (File)
Tue, Feb 18, 10:11 PM
Unknown Object (File)
Sat, Feb 8, 3:21 AM
Unknown Object (File)
Sat, Feb 8, 3:21 AM
Unknown Object (File)
Sat, Feb 8, 3:21 AM
Unknown Object (File)
Sat, Feb 8, 3:21 AM
Subscribers

Details

Summary

Ref T7199. Two notable changes:

  • Process multiple commands.
  • Process commands when creating or updating a task.

And generally clean things up a bit.

Test Plan
  • Used receive-test to execute all commands for new tasks.
  • Used receive-test to execute all commands for existing tasks.
  • Used a combination of commands to produce varied effects.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Allow Maniphest to process multiple commands via email.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
  • Restore "CC" rule.
src/applications/maniphest/mail/ManiphestReplyHandler.php
130–136

Oh, I accidentally removed this rule. I'll restore it. I think all these reply handlers are going to start sharing a lot more code real soon.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 1 2015, 12:23 AM
This revision was automatically updated to reflect the committed changes.