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
F19812090: D19995.diff
Wed, Mar 4, 3:37 AM
F19803710: D19995.id47735.diff
Mon, Mar 2, 1:24 AM
F19556636: D19995.diff
Jan 30 2026, 8:32 PM
F19556620: D19995.diff
Jan 30 2026, 8:27 PM
F19543717: D19995.diff
Jan 22 2026, 10:49 PM
F19532908: D19995.diff
Jan 20 2026, 2:18 PM
F19532903: D19995.diff
Jan 20 2026, 2:16 PM
F19532836: D19995.diff
Jan 20 2026, 1:50 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.