Skip to content

feat: ability to add recovery email #616

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

Merged
merged 11 commits into from
Jul 27, 2024
Merged

Conversation

benjaminshafii
Copy link
Collaborator

@benjaminshafii benjaminshafii commented Jul 27, 2024

What does this PR do?

This PR adds functionality for setting and verifying a recovery email for user accounts. It includes the following changes:

  1. Adds new fields to the accounts table for storing the hashed recovery email and recovery email verification timestamp.
  2. Implements new API endpoints for setting and verifying the recovery email.
  3. Creates email templates for sending recovery email confirmation.
  4. Adds a new page for verifying the recovery email.
  5. Updates the user security settings page to include a section for managing the recovery email.
  6. Implements a new email sending function that can be toggled to log emails to the console for testing purposes.

Note: Does not include the actual recovery of the account

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Checklist

Required

  • Read Contributing Guide
  • Self-reviewed my own code
  • Tested my code in a local environment
  • Commented on my code in hard-to-understand areas
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the UnInbox Docs if changes were necessary

Demo + explanation

new-recovery-email.mp4

Recovery Email Screenshot

Screenshot 2024-07-27 at 11 49 57

@benjaminshafii benjaminshafii changed the title Ability to add recovery email feat: ability to add recovery email Jul 27, 2024
@BlankParticle BlankParticle force-pushed the ability-to-add-recovery-email branch from b3312c3 to d0f88f0 Compare July 27, 2024 12:00
@BlankParticle BlankParticle merged commit b254c46 into main Jul 27, 2024
5 checks passed
@BlankParticle BlankParticle deleted the ability-to-add-recovery-email branch July 28, 2024 04:53
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