Paths

Table of Contentst

Differential D12234

Allow Maniphest to process multiple commands via email
ClosedPublic
Actions

Authored by epriestley on Apr 1 2015, 12:17 AM.

Tags

None

Referenced Files

	F15462667: D12234.id29393.diff
	Tue, Apr 1, 4:36 PM

	F15447270: D12234.id.diff
	Thu, Mar 27, 10:49 PM

	F15421841: D12234.id29393.diff
	Sat, Mar 22, 2:35 AM

	F15420789: D12234.id29391.diff
	Fri, Mar 21, 5:34 PM

	F15419112: D12234.id29390.diff
	Fri, Mar 21, 3:34 AM

	F15418004: D12234.id.diff
	Thu, Mar 20, 8:06 PM

	F15413944: D12234.diff
	Wed, Mar 19, 10:06 PM

	F15411820: D12234.id29391.diff
	Wed, Mar 19, 9:55 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T7199: Expand the power of "!action" commands in email

Commits

Restricted Diffusion Commit
rP77b23035c3c2: Allow Maniphest to process multiple commands via email

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 29390.Apr 1 2015, 12:17 AM

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.

epriestley added a task: T7199: Expand the power of "!action" commands in email.

Herald added a subscriber: epriestley. · View Herald TranscriptApr 1 2015, 12:17 AM

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 accepted this revision.Apr 1 2015, 12:23 AM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Apr 1 2015, 12:23 AM

Closed by commit rP77b23035c3c2: Allow Maniphest to process multiple commands via email (authored by epriestley, committed by epriestley). · Explain WhyApr 1 2015, 12:25 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

mail/

ManiphestReplyHandler.php

138 lines

Diff 29393

src/applications/maniphest/mail/ManiphestReplyHandler.php

Loading...