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.

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

Forget Password #104

Merged
merged 19 commits into from
Sep 23, 2024
Merged

Forget Password #104

merged 19 commits into from
Sep 23, 2024

Conversation

wr1159
Copy link

@wr1159 wr1159 commented Sep 17, 2024 •
edited
Loading

Resolves #33, #35

Changes

  • Backend
    • Add 2 new routes
      • forget password to send email with token and add to DB the token and expiry time
      • reset password to reset password with toke and remove passwordReset item
    • Add new password reset model with email, token, expiryTime
    • install nodemailer to send email
  • Frontend
    • forget password page
    • reset password page

Demo

resetpassword.mp4

SelwynAng and others added 16 commits September 15, 2024 12:08
@SelwynAng
Set up static user settings page
17101da
@SelwynAng
Enable fetching of user data from user service backend to display in …
ac658b0 
…user settings
@SelwynAng
Add skillLevel to user model in user service and enable updates to sk…
4b46575 
…illLevel from frontend
@TanPingZhi
Connect frontend to backend's sendPasswordResetEmail
d9c44a0
@wr1159
Merge branch 'main' into feature/user-service/forget-password
1835b52
@wr1159
Refactor forget password frontend and add links to forget password
bf07cdc
@wr1159
Prettier format
b81fde1
@wr1159
Prettier: fix
06aefba
@wr1159
Add database saving for password reset
d5ddcdb
@wr1159
Add partial route to send email
2551002
@wr1159
Add mailing service
0acddd0
@wr1159
Integrate with forget password link
b6dc1ed
@wr1159
Add reset-password frontend and backend
7047c9f
@wr1159
Fix reset password backend endpoint
455029a
@wr1159
Add reset password functionality
36ad1f4
@wr1159
Fix status code and add to README.md
4b12271
@wr1159 wr1159 self-assigned this Sep 17, 2024
@wr1159 wr1159 linked an issue Sep 17, 2024 that may be closed by this pull request
F1.6.2 The service shall allow users to navigate to the reset password page from the reset password link to reset their password. #35
Closed
@jq1836 jq1836 changed the base branch from main to chore/add-user-interface September 23, 2024 18:11
@jq1836 jq1836 changed the base branch from chore/add-user-interface to main September 23, 2024 18:11
TanPingZhi
TanPingZhi approved these changes Sep 23, 2024
View reviewed changes
Copy link

@TanPingZhi TanPingZhi left a comment •
edited
Loading

Choose a reason for hiding this comment

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

It works but theres some minor issues that doesnt affect functionality. Will make bug issue #123

@TanPingZhi TanPingZhi merged commit 558d135 into main Sep 23, 2024
@TanPingZhi TanPingZhi deleted the feature/user-service/forget-password branch September 23, 2024 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TanPingZhi TanPingZhi TanPingZhi approved these changes

Assignees

@wr1159 wr1159

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@wr1159 @TanPingZhi @SelwynAng