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

Feat Enable Password Reset #318

Merged
merged 7 commits into from
Mar 5, 2025
Merged

Feat Enable Password Reset #318

merged 7 commits into from
Mar 5, 2025

Conversation

khalifan-kfan
Copy link
Collaborator

Description

This PR adds a feature to reset the password using superbase

Discussion

Closes: #309

Preview

Link to app preview if relevant

How to test it ( include screenshots)

  1. Run this branch locally
  2. Make sure the server is running
  3. Go to the dashboard
  4. Signout
Screenshot 2024-08-06 at 21 24 40
  1. Create a new user if you dont have one
Screenshot 2024-08-06 at 21 24 48
  1. Then for the user you have created initiate a reset password with there email
Screenshot 2024-08-06 at 21 24 54
  1. An email will not be sent locally but rather you will find it under monitoring on the link:
    http://localhost:54324/ among the running docker servicer, it will be under monitoring.
Screenshot 2024-08-06 at 21 31 25 Screenshot 2024-08-06 at 19 36 06
  1. Click on the link to reset password and you will be redirected to a screen where you can enter the new password
Screenshot 2024-08-06 at 21 20 52
  1. On submission, you will be done reseting your password.

@github-actions github-actions bot added the App: Dashboard Updates related to Dashboard app label Aug 6, 2024
Copy link

nx-cloud bot commented Aug 6, 2024

View your CI Pipeline Execution ↗ for commit b1f7c73.

Command Status Duration Result
nx build ✅ Succeeded 5m 24s View ↗
nx run-many --target=lint ✅ Succeeded 6s View ↗

☁️ Nx Cloud last updated this comment at 2025-03-04 22:27:30 UTC

Copy link
Collaborator

@chrismclarke chrismclarke left a comment

Choose a reason for hiding this comment

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

Thanks @khalifan-kfan
Apologies for the slow review, all working well when I tested locally (thanks for sharing the link to check supabase inbucket to see preview emails). I'll merge and deploy to production to see if any extra configuration needed on the supabase end to send emails

@chrismclarke chrismclarke merged commit 0753a26 into main Mar 5, 2025
3 checks passed
@chrismclarke chrismclarke deleted the ft-password-reset branch March 5, 2025 00:08
@chrismclarke
Copy link
Collaborator

Just tested on live site and I was successfully able to reset my password so all looking good, thanks for the addition

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App: Dashboard Updates related to Dashboard app
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

feat(dashboard): Password reset
2 participants