Changeset View
Changeset View
Standalone View
Standalone View
src/applications/conpherence/editor/ConpherenceEditor.php
Show First 20 Lines • Show All 232 Lines • ▼ Show 20 Lines | protected function addEmailPreferenceSectionToMailBody( | ||||
array $xactions) { | array $xactions) { | ||||
$href = PhabricatorEnv::getProductionURI( | $href = PhabricatorEnv::getProductionURI( | ||||
'/'.$object->getMonogram().'?settings'); | '/'.$object->getMonogram().'?settings'); | ||||
$label = pht('EMAIL PREFERENCES FOR THIS ROOM'); | $label = pht('EMAIL PREFERENCES FOR THIS ROOM'); | ||||
$body->addLinkSection($label, $href); | $body->addLinkSection($label, $href); | ||||
} | } | ||||
protected function getMailSubjectPrefix() { | protected function getMailSubjectPrefix() { | ||||
return PhabricatorEnv::getEnvConfig('metamta.conpherence.subject-prefix'); | return pht('[Conpherence]'); | ||||
} | } | ||||
amckinley: Just for giggles, why don't we implement `getMailSubjectPrefix` in… | |||||
epriestleyAuthorUnsubmitted Done Inline ActionsI'd probably like to get there eventually. Right now, it would mean:
I don't think either one is a big deal, but this moves us toward that without actually breaking anything. (Some applications also have more than one object, e.g. SSHKeys send [SSH Key] rather than [Auth].) The whole feature is also semi-mooted by the application(...) mail stamp. epriestley: I'd probably like to get there eventually. Right now, it would mean:
- You can't… | |||||
protected function supportsSearch() { | protected function supportsSearch() { | ||||
return true; | return true; | ||||
} | } | ||||
} | } |
Just for giggles, why don't we implement getMailSubjectPrefix in PhabricatorApplicationTransactionEditor and just have it do something like this: