Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Conversation

@babolivier
Copy link
Contributor

@babolivier babolivier commented Mar 11, 2022

Fixes #12205

cc @reivilibre

@babolivier babolivier requested a review from a team as a code owner March 11, 2022 10:16
@babolivier
Copy link
Contributor Author

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).

@babolivier babolivier requested a review from richvdh March 11, 2022 10:41
@babolivier babolivier requested a review from richvdh March 11, 2022 12:27
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm otherwise

@babolivier
Copy link
Contributor Author

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.

@babolivier babolivier enabled auto-merge (squash) March 11, 2022 13:03
@babolivier babolivier merged commit 003cc69 into develop Mar 11, 2022
@babolivier babolivier deleted the babolivier/username_template branch March 11, 2022 13:20
Copy link
Contributor

@nadonomy nadonomy left a 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>
Copy link
Contributor

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tweak Sign In With Apple/SSO copy to make required data explicit

4 participants