Homec4science

Maniphest - add support for !assign command

Authored by Bob Trahan <bob.trahan@gmail.com> on Oct 14 2013, 21:29.

Description

Maniphest - add support for !assign command

Summary:
also try to centralize some of the command parsing logic. note that differential is still an exception here. it uses a whitelist-style regex. i think long-term we should have this for every app but changing it seemed too big for this diff.

Fixes T3937.

Test Plan:
echo '!assign btrahan' | ./bin/mail receive-test --as xerxes --to T22 ; echo '!claim' | ./bin/mail receive-test --as xerxes --to T22

unit tests passed, though my new one is silly

Reviewers: epriestley

Reviewed By: epriestley

CC: Korvin, epriestley, aran

Maniphest Tasks: T3937

Differential Revision: https://secure.phabricator.com/D7307

Details

Committed
Bob Trahan <bob.trahan@gmail.com>Oct 14 2013, 21:29
Pushed
aubortJan 31 2017, 17:16
Parents
rPH13178ec2792b: Prepare the policy rule edit endpoint for integration
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPHd0127f95e541: Maniphest - add support for !assign command (authored by Bob Trahan <bob.trahan@gmail.com>).Oct 14 2013, 21:29