revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-02-14 15:58:39 +01:00
Code Issues Releases Wiki Activity
phorge-phorge/src/applications/maniphest/replyhandler/__init__.php

21 lines
748 B
PHP
Raw Normal View History

Support email multiplexing for private Reply-To addresses Summary: Provide a base PhabricatorMailReplyHandler class which handles the plumbing for multiplexing email if necessary and supporting public and private reply handler addressses. DifferentialReplyHandler now extends it, and a new ManiphestReplyHandler also does. The general approach here is that we have three supported cases: - no reply handler, default config, same as what we're doing now - public reply handler, requires overriding classes but just sets "reply-to" to some address the install generates and still sends only one email - private reply handler, provides a default generation mechanism or you can override it and splits mail apart so we send one to each recipient Test Plan: Sent email from Maniphest and Differential with and without reply-handler-domains set. Reviewed By: aran Reviewers: jungejason, tuomaspelkonen, aran CC: aran, epriestley Differential Revision: 254
2011-05-09 16:31:26 -07:00
<?php
/**
* This file is automatically generated. Lint this module to rebuild it.
* @generated
*/
Make reply-to fully work in Maniphest and Differential for open source Phabricator Summary: Hook up the last pieces. This shouldn't impact the Facebook install, EXCEPT that I removed "!accept" and added "!rethink" (plan changes). If you want to continue supporting !accept, you should override the method in your subclass if you don't already. Test Plan: Used the Mail Receiver test console to send mail to tasks and revisions. Reviewed By: aran Reviewers: jungejason, tuomaspelkonen, aran CC: aran Differential Revision: 289
2011-05-16 12:31:18 -07:00
phutil_require_module('phabricator', 'applications/maniphest/constants/status');
phutil_require_module('phabricator', 'applications/maniphest/constants/transactiontype');
phutil_require_module('phabricator', 'applications/maniphest/editor/transaction');
phutil_require_module('phabricator', 'applications/maniphest/storage/transaction');
Support email multiplexing for private Reply-To addresses Summary: Provide a base PhabricatorMailReplyHandler class which handles the plumbing for multiplexing email if necessary and supporting public and private reply handler addressses. DifferentialReplyHandler now extends it, and a new ManiphestReplyHandler also does. The general approach here is that we have three supported cases: - no reply handler, default config, same as what we're doing now - public reply handler, requires overriding classes but just sets "reply-to" to some address the install generates and still sends only one email - private reply handler, provides a default generation mechanism or you can override it and splits mail apart so we send one to each recipient Test Plan: Sent email from Maniphest and Differential with and without reply-handler-domains set. Reviewed By: aran Reviewers: jungejason, tuomaspelkonen, aran CC: aran, epriestley Differential Revision: 254
2011-05-09 16:31:26 -07:00
phutil_require_module('phabricator', 'applications/metamta/replyhandler/base');
When an email reply to a task includes files, attach them to the task Summary: Allow files to be attached to a task by attaching them to an email reply to the task. Test Plan: Applied this patch live since I haven't managed to get inbound email configured locally, then attached files to a task via email. Reviewed By: jungejason Reviewers: tuomaspelkonen, jungejason, aran CC: anjali, aran, jungejason Differential Revision: 369
2011-05-29 03:19:06 -07:00
phutil_require_module('phabricator', 'applications/phid/constants');
Support email multiplexing for private Reply-To addresses Summary: Provide a base PhabricatorMailReplyHandler class which handles the plumbing for multiplexing email if necessary and supporting public and private reply handler addressses. DifferentialReplyHandler now extends it, and a new ManiphestReplyHandler also does. The general approach here is that we have three supported cases: - no reply handler, default config, same as what we're doing now - public reply handler, requires overriding classes but just sets "reply-to" to some address the install generates and still sends only one email - private reply handler, provides a default generation mechanism or you can override it and splits mail apart so we send one to each recipient Test Plan: Sent email from Maniphest and Differential with and without reply-handler-domains set. Reviewed By: aran Reviewers: jungejason, tuomaspelkonen, aran CC: aran, epriestley Differential Revision: 254
2011-05-09 16:31:26 -07:00
phutil_require_module('phabricator', 'infrastructure/env');
Make reply-to fully work in Maniphest and Differential for open source Phabricator Summary: Hook up the last pieces. This shouldn't impact the Facebook install, EXCEPT that I removed "!accept" and added "!rethink" (plan changes). If you want to continue supporting !accept, you should override the method in your subclass if you don't already. Test Plan: Used the Mail Receiver test console to send mail to tasks and revisions. Reviewed By: aran Reviewers: jungejason, tuomaspelkonen, aran CC: aran Differential Revision: 289
2011-05-16 12:31:18 -07:00
phutil_require_module('phutil', 'utils');
Support email multiplexing for private Reply-To addresses Summary: Provide a base PhabricatorMailReplyHandler class which handles the plumbing for multiplexing email if necessary and supporting public and private reply handler addressses. DifferentialReplyHandler now extends it, and a new ManiphestReplyHandler also does. The general approach here is that we have three supported cases: - no reply handler, default config, same as what we're doing now - public reply handler, requires overriding classes but just sets "reply-to" to some address the install generates and still sends only one email - private reply handler, provides a default generation mechanism or you can override it and splits mail apart so we send one to each recipient Test Plan: Sent email from Maniphest and Differential with and without reply-handler-domains set. Reviewed By: aran Reviewers: jungejason, tuomaspelkonen, aran CC: aran, epriestley Differential Revision: 254
2011-05-09 16:31:26 -07:00
phutil_require_source('ManiphestReplyHandler.php');
Copy permalink