Open
Conversation
Merge: Forgot Password functionality to main
# Conflicts: # package-lock.json # src/App.tsx
E2604 password reset
API_BASE_URL is pulled in multiple locations, by setting this as a contant in a centralized location other pages can pull in this constant This gives us one sourcce of truth for the API_BASE_URL rather than having it scattered in multiple areas Added additional information in the readme file on where to set env variables for docker compose Additionally, included a sample.env file which can be used to show what to include in the env file and how to get docker to load it Added .env to the gitignore list
Centralizing the Api contant
…on, bookmarking, or expired tokens.
We can't use import.meta here we need to load the env first and then call env varsity.
… and ResetPassword.
…ts_pt_1' into front_end_refinements_pt_1
Standardize ForgotPassword and ResetPassword pages with Login.tsx conventions
Adding tests to cover ForgotPassword with vitest
…e initial PW reset tests back in.
Vitest Coverage For Reset Password Front-
johnmweisz
changed the title
johnmweisz
commented
Mar 22, 2026
johnmweisz
commented
Mar 22, 2026
…padding above the container Changing from mt-xxl-5 to mt-5 resolves the issue and keeps a top margin of 5 This change also needed to be done on the Login page
When viewing the login/forgot/reset pages the top padding is off
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This implements the frontend portion of password reset functionality.
Backend PR is here: expertiza/reimplementation-back-end#302
Feature detail can be reviewed here: https://wiki.expertiza.ncsu.edu/index.php?title=CSC/ECE_517_Spring_2026_-_E2604._Finish_Password_Resets
Issue: #264
Notes