feat: implement password management web updates#419
feat: implement password management web updates#419MillicentAmolo wants to merge 1 commit intothoth-tech:9.xfrom
Conversation
AB-Deakin
left a comment
There was a problem hiding this comment.
I've ran tests on the frontend changest and it is working correctly and code looks good. With these changes coming off the development branch instead of 9.x it may cause some issues as well. May need to check with the senior team if it can merge as is or if it needs to be branched off of the 9.x branch like the API PR. But everything else works and looks good from my testing.


ibi420
left a comment
There was a problem hiding this comment.
Hey @MillicentAmolo , great work on this. I can confirm the functionality works as expected. Despite the earlier issue with creating the function on the development branch instead of 9.x, everything now runs smoothly without errors. You’ve adhered to the TypeScript migration pattern and made good use of modern Angular components. The UI is consistent with OnTrack’s look and feel. Excellent job on this.
|
Great work on this PR, @MillicentAmolo. I pulled the branch down and tested the new forgot/reset password flow. Functionality works as expected across both Chrome and Firefox, navigation between login, forgot password, and reset password is smooth and error-free. From a technical standpoint, you’ve followed the TypeScript migration pattern well and used modern Angular components cleanly. The UI is consistent with OnTrack’s design language, and your updates to the authentication service and routing are easy to follow. Code readability is strong, with comments in the right places. Nice Job |

Description
Added forgot/reset password functionality to the application. This includes new components for handling password recovery and updates to existing authentication and state management files to integrate the feature.
Fixes # (issue)
Implement forgot/reset password feature
Type of change
New feature (non-breaking change which adds functionality)
How Has This Been Tested?
Created new forgot-password and reset-password components.
Updated authentication.service.ts to handle password reset requests.
Updated routing in doubtfire.states.ts to include forgot/reset password states.
Confirmed navigation between login, forgot password, and reset password flows works correctly.
Testing Checklist:
Tested in latest Chrome
Tested in latest Firefox
Checklist:
[x ] My code follows the style guidelines of this project
[x ] I have performed a self-review of my own code
[ x] I have commented my code in hard-to-understand areas
I have made corresponding changes to the documentation
[x ] My changes generate no new warnings
I have requested a review from @macite and @jakerenzella on the Pull Request