Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edit Contents of Confirmation Instructions Email #952

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

aaronskiba
Copy link
Collaborator

Changes proposed in this PR:

  • The upcoming 4.1.1+portage-4.2.3 release will require all existing users to confirm their emails. This commit edits the contents of that confirmation email. "Thank you for registering." makes sense for users that just created an account, but doesn't sound right when read by users with existing accounts. Thus, we are removing the sentence.

The upcoming `4.1.1+portage-4.2.3` release will require all existing users to confirm their emails. This commit edits the contents of that confirmation email. "Thank you for registering." makes sense for users that just created an account, but doesn't sound right when read by users with existing accounts. Thus, we are removing the sentence.
Copy link
Collaborator

@lagoan lagoan left a comment

Choose a reason for hiding this comment

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

LGTM!

@aaronskiba aaronskiba merged commit a4a9d25 into deployment-portage Nov 20, 2024
12 checks passed
@aaronskiba aaronskiba deleted the aaron/edit-email-confirmation-text branch November 20, 2024 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants