Homec4science

Conpherence - reply handler integration

Authored by Bob Trahan <bob.trahan@gmail.com> on Jan 26 2013, 01:03.

Description

Conpherence - reply handler integration

Summary: Added a reply handler. A few problems -- first, I can't seem to get this to actually send me email so I haven't been able to reply (which I would have done by generating a reply, then copying the raw email into scripts/mail_handler.php). Second, the subject is often terrible on these emails -- unless the conpherence is named its something gross like "E4:" Third, on create I am noticing an error on array_combine() which I think is related to the need to write array_combine_not_broken or what have you I saw go by... (PhabricatorTransactionEditor does array_combine(xaction->getOldValue(), xaction->getOldValue()) and complains that the arrays are empty)

Test Plan: noted that /mail/ said mails were being sent

Reviewers: epriestley

Reviewed By: epriestley

CC: chad, aran, Korvin

Maniphest Tasks: T2399

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

Details

Committed
Bob Trahan <bob.trahan@gmail.com>Jan 26 2013, 01:03
Pushed
aubortJan 31 2017, 17:16
Parents
rPHe344f57df116: Conpherence CSS tweaks.
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPH4d22c9104f16: Conpherence - reply handler integration (authored by Bob Trahan <bob.trahan@gmail.com>).Jan 26 2013, 01:03