Skip to content

Allow user to add multiple emails #48

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.

Sign up for GitHub

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

Jump to bottom
Merged
Mariatta merged 4 commits into from
Mar 28, 2025
Merged

Allow user to add multiple emails #48

Mariatta merged 4 commits into from
Mar 28, 2025

Conversation

@Pradhvan
Copy link
Contributor

@Pradhvan Pradhvan commented Mar 28, 2025 •
edited
Loading

  • User can have max 3 emails.
  • Updated Change Email address button -> Manage Emails for more generic
    email actions.
  • Updated HTML templates to reflect Change Email address -> Manage Emails.

Closes #44

Changes 👇🏾

image

image

Pradhvan and others added 4 commits March 28, 2025 19:10
@Pradhvan
Allow user to add multiple emails
e820314 
- User can have max 3 emails.
- Updated Change Email address button -> Manage Emails for more generic
  email actions.
- Updated HTML templates to reflect Change Email address -> Manage Emails.

Closes pyladies#44
@Pradhvan
Revert base template change
8faae4a
@Mariatta
Merge branch 'main' into Issue-44
fa1ac8a
@Mariatta
add back ACCOUNT_MAX_EMAIL_ADDRESSES setting
f5827dd
Mariatta
Mariatta approved these changes Mar 28, 2025
View reviewed changes
Copy link
Member

@Mariatta Mariatta left a comment

Choose a reason for hiding this comment

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

This is great! Thank you!

@Mariatta Mariatta merged commit 178d592 into pyladies:main Mar 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Mariatta Mariatta Mariatta approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Allow user to add alternate emails

2 participants

@Pradhvan @Mariatta