Page MenuHomePhabricator

Use the customizable "Welcome Mail" message in welcome mail
ClosedPublic

Authored by epriestley on Jan 18 2019, 3:08 AM.
Tags
None
Referenced Files
F19507916: D19995.id47729.diff
Fri, Jan 9, 11:57 PM
F19507915: D19995.id.diff
Fri, Jan 9, 11:56 PM
F19502835: D19995.id47735.diff
Thu, Jan 8, 7:54 PM
F19502676: D19995.id47729.diff
Thu, Jan 8, 6:14 PM
F19225298: D19995.diff
Wed, Dec 17, 3:57 PM
F19082434: D19995.diff
Dec 2 2025, 11:07 AM
F19056075: D19995.id47735.diff
Nov 28 2025, 6:56 PM
F19056006: D19995.id47735.diff
Nov 28 2025, 6:51 PM
Subscribers

Details

Summary

Depends on D19994. See PHI1027. If an install has customized the "Welcome Mail" message, include it in welcome mail. A special custom message from the profile screen overrides it, if provided.

(I fiddled with putting the custom message as "placeholder" text in the remarkup area as a hint, but newlines in "placeholder" text appear to have issues in Safari and Firefox. I think this is probably reasonably clear as-is.)

Make both render remarkup-into-text so things like links work properly, as it's reasonably likely that installs will want to link to things.

Test Plan
  • With custom "Welcome Mail" text, sent mail with no custom override (got custom text) and a custom override (got overridden text).
  • Linked to some stuff, got sensible links in the mail (bin/mail show-outbound).

Diff Detail

Repository
rP Phabricator
Branch
welcome6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21587
Build 29427: Run Core Tests
Build 29426: arc lint + arc unit

Event Timeline

  • When the mail body has a global default, make the reminder text link to it.
This revision is now accepted and ready to land.Jan 19 2019, 12:51 AM
This revision was automatically updated to reflect the committed changes.
avivey added inline comments.
src/applications/people/controller/PhabricatorPeopleWelcomeController.php
51

$default_message may return null under some case - maybe any fresh install/upgrade? Is there a default-default value?

src/applications/people/controller/PhabricatorPeopleWelcomeController.php
51

Fixed by D20001.