Page MenuHomePhabricator

Add "Auth Messages" to support customizing onboarding/welcome flows
ClosedPublic

Authored by epriestley on Jan 17 2019, 7:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 20, 7:37 AM
Unknown Object (File)
Fri, Nov 15, 9:49 PM
Unknown Object (File)
Thu, Nov 14, 3:56 AM
Unknown Object (File)
Tue, Nov 5, 6:45 PM
Unknown Object (File)
Tue, Nov 5, 8:04 AM
Unknown Object (File)
Thu, Oct 24, 10:27 PM
Unknown Object (File)
Oct 18 2024, 7:12 PM
Unknown Object (File)
Oct 16 2024, 4:38 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13222. Long ago, we had a Config option (welcome.html) to let you dump HTML onto the login screen, but this was relatively hard to use and not good from a security perspective.

In some cases this was obsoleted by Dashboards, but there's at least some remaining set of use cases for actual login instructions on the login screen. For example, WMF has some guidance on which SSO mechanism to use based on what types of account you have. On secure, users assume they can register by clicking "Log In With GitHub" or whatever, and it might reduce frustration to tell them upfront that registration is closed.

Some other types of auth messaging could also either use customization or defaults (e.g., the invite/welcome/approve mail).

We could do this with a bunch of Config options, but I'd generally like to move to a world where there's less stuff in Config and more configuration is contextual. I think it tends to be easier to use, and we get a lot of fringe benefits (granular permissions, API, normal transaction logs, more abililty to customize workflows and provide contextual help/hints, etc). Here, for example, we can provide a remarkup preview, which would be trickier with Config.

This does not actually do anything yet.

Test Plan

Screen Shot 2019-01-17 at 11.29.07 AM.png (933×1 px, 184 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.Jan 17 2019, 7:43 PM
amckinley added inline comments.
src/applications/auth/message/PhabricatorAuthWelcomeMailMessageType.php
4

Should this replace the one-off custom messages from D19991?

This revision is now accepted and ready to land.Jan 19 2019, 12:48 AM

My thinking here is that both types are useful -- you can have a generic "here's how to get set up", and then override it with alternate instructions if it's a special case.

Or if you have three different weird ways that external users interact with invites, you can put the three different weird variations on a wiki somewhere and copy/paste them in appropriately. It sounds like the use case in PHI1027 might possibly benefit from that kind of flexibility.

This revision was automatically updated to reflect the committed changes.