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

Account emails #293

Merged
merged 21 commits into from
Sep 10, 2024
Merged

Account emails #293

merged 21 commits into from
Sep 10, 2024

Conversation

tnix100
Copy link
Member

@tnix100 tnix100 commented Aug 20, 2024

Adds the ability for accounts to have an email address for password recovery and important alerts.

May also be used in the future for anti-spam mechanisms, such as detecting ban evasion and requiring email verification for suspicious accounts. But those are out of scope of this PR.

  • HTML email templates
  • Text email templates
  • The ability for emails to be rendered and sent
  • Create new token system (our current one sucks and won't work for email actions)
  • The ability for email addresses to be added to accounts
  • The ability for email addresses to be verified
  • The ability for account passwords to be reset using an account's verified email address
  • The ability for account security actions to be reverted and for the account to be locked when a suspicious change happens
  • Send password change email alerts
  • Send multi-factor authenticator adding/removing email alerts

Some of these will require frontend changes in meower-media/website. Email-related pages will be stored under emails in the website repo.

@tnix100 tnix100 added enhancement New feature or request High Priority Needs attention, but not urgent labels Aug 20, 2024
@tnix100 tnix100 self-assigned this Aug 20, 2024
@tnix100 tnix100 marked this pull request as ready for review September 9, 2024 13:18
Copy link

@HAZARD6896 HAZARD6896 left a comment

Choose a reason for hiding this comment

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

yippee

@tnix100
Copy link
Member Author

tnix100 commented Sep 10, 2024

I need to add the ability for email addresses to be removed before this gets merged.

@tnix100 tnix100 merged commit 69176d2 into develop Sep 10, 2024
1 check passed
@tnix100 tnix100 deleted the tnix-emails branch September 10, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request High Priority Needs attention, but not urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants