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

Add parent email confirmation when added to a student account #34015

Merged
merged 6 commits into from Apr 3, 2020

Conversation

bethanyaconnor
Copy link
Contributor

Preview can be seen at http://localhost-studio.code.org:3000/rails/mailers/parent_mailer/parent_email_added_to_student_account_preview

Links

Testing story

Reviewer Checklist:

  • Tests provide adequate coverage
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@dju90
Copy link
Contributor

dju90 commented Apr 3, 2020

Email body looks good, did a double read-through to check for typos

@@ -250,6 +250,7 @@ en:
%{name}
parent_mailer:
student_associated_subject: 'Login information for Code.org'
parent_email_added_subject: 'Welcome to Code.org'
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add an exclamation point to the end of the subject? 'Welcome to Code.org!' (same as the teacher email)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@bethanyaconnor bethanyaconnor marked this pull request as ready for review April 3, 2020 16:51
@bethanyaconnor bethanyaconnor requested a review from a team as a code owner April 3, 2020 16:51
Copy link
Contributor

@Hamms Hamms left a comment

Choose a reason for hiding this comment

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

couple nits, otherwise LGTM!


%p
Don’t have a computer? We can suggest some options that work great on mobile phones or tablets too!
%a{href:"https://code.org/athom#apps"}See smartphone options.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
%a{href:"https://code.org/athom#apps"}See smartphone options.
%a{href:"https://code.org/athome#apps"}See smartphone options.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice catch!

end

puts classes
Copy link
Contributor

Choose a reason for hiding this comment

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

stray debug statement

@bethanyaconnor bethanyaconnor merged commit d02ac31 into staging Apr 3, 2020
@bethanyaconnor bethanyaconnor deleted the parent-email-confirmation branch April 3, 2020 21:05
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.

None yet

3 participants