-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Update the SSO username picker template to comply with SIWA guidelines #12210
Conversation
|
Would also like a check from @nadonomy on the copy before it goes in (looks like I can't request reviews from Nad on this repo). |
richvdh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm otherwise
|
I've had a chat out-of-bound with @nadonomy and it sounds like the changes are fine, so let's merge it and get it deployed it soon. |
nadonomy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@babolivier @callahad sorry PEBKAC on my behalf for not submitting this before. Would be great to get this right given the pedantry at review time.
| <header> | ||
| <h1>Your account is nearly ready</h1> | ||
| <p>Check your details before creating an account on {{ server_name }}</p> | ||
| <h1>Choose your user name</h1> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@babolivier Per the issue this should be:
Choose your account name
Fixes #12205
cc @reivilibre