Conversation
…o feat/forgot-password
…o feat/forgot-password
There was a problem hiding this comment.
Pull request overview
This PR adds forgot password and reset password functionality to the authentication library, enabling users to request password reset links and set new passwords via token-based verification.
Changes:
- Added two new pages:
ForgotPasswordPageandResetPasswordPagewith corresponding routes - Updated all React Router imports from
react-routertoreact-router-domfor consistency - Enhanced the Vite build configuration to support UMD format and define external dependency globals
- Modified the sign-in page to include a "Forgot Password" link
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| vite.config.ts | Added UMD build format and global definitions for external dependencies |
| src/providers/AuthProvider.tsx | Added routes for forgot/reset password pages and updated import |
| src/pages/auth/SignInPage.tsx | Replaced forgot password button with link to new page |
| src/pages/auth/SignUpPage.tsx | Updated React Router import to use react-router-dom |
| src/pages/auth/ResetPasswordPage.tsx | New page for resetting password with token validation |
| src/pages/auth/ForgotPasswordPage.tsx | New page for requesting password reset link |
| src/components/RequirePermissions.tsx | Updated React Router import to use react-router-dom |
| package.json | Incremented version to 1.0.6 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| import { useT } from "@ciscode/ui-translate-core"; | ||
| import { useNavigate, useLocation } from "react-router"; | ||
| import { useNavigate, useLocation } from "react-router-dom"; | ||
| import { Link } from "react-router-dom" |
There was a problem hiding this comment.
Missing semicolon at end of import statement. Add semicolon for consistency with other imports in the file.
Suggested change
| import { Link } from "react-router-dom" | |
| import { Link } from "react-router-dom"; |
| {t("ForgotPasswordPage.subtitle", { defaultValue: "Enter your email to receive a reset link." })} | ||
| </p> | ||
|
|
||
| {error && <InlineError message={error} />} |
There was a problem hiding this comment.
Trailing whitespace detected at end of line. Remove the extra space after the closing JSX tag.
Suggested change
| {error && <InlineError message={error} />} | |
| {error && <InlineError message={error} />} |
Zaiidmo
approved these changes
Jan 30, 2026
Zaiidmo
added a commit
that referenced
this pull request
Feb 2, 2026
* parents translate, components receive plain strings * 1.0.3 * tested in local, bug fixed * 1.0.4 * 1.0.5 * forgot and reset password done * 1.0.6 * Feat/forgot password (#8) * parents translate, components receive plain strings * 1.0.3 * tested in local, bug fixed * 1.0.4 * forgot and reset password done * 1.0.6 * Docs: Add Export Strategy and Workflow Guidelines (#9) * docs: add copilot development instructions - Add React component library development guidelines - Include accessibility and i18n requirements - Document component patterns and testing standards - Add release checklist * docs: create task documentation structure for UI module - Add active tasks folder with UI-MODULE- prefix - Establish workflow for UI component tasks - Add UI-specific documentation focus (accessibility, responsive) * docs: add export strategy and workflow guidelines - Define export strategy (Components + Hooks + Types only) - Document rationale for public API exports - Add workflow with UI-MODULE- prefix convention - Include breaking changes handling for component APIs - Add task documentation structure examples * 1.0.7 * Updated workflows (#10) * ops: updated release pipeline to run only on version changes, and created ci workflow * 1.0.8 * docs(workflow): add Git Flow and npm version requirements (#11) - Add Git Flow branching strategy (develop/master) - Document npm version command before push - Add prepublishOnly hook recommendation - Update workflow with proper branch management - Clear warnings about PR targeting Co-authored-by: Reda Channa <r.channa@ciscod.com> Co-authored-by: Zaiid Moumni <141942826+Zaiidmo@users.noreply.github.com> * docs: added different documentations * 1.0.9 * ops: updated publishing trigger * verify email page done * merged * 1.0.10 --------- Co-authored-by: Zaiidmo <zaiidmoumnii@gmail.com> Co-authored-by: RedaChannaCiscode <r.channa@ciscod.com> Co-authored-by: Zaiid Moumni <141942826+Zaiidmo@users.noreply.github.com> Co-authored-by: Ciscode-Admin <info@ciscod.com>
Zaiidmo
added a commit
that referenced
this pull request
Feb 2, 2026
* parents translate, components receive plain strings * 1.0.3 * tested in local, bug fixed * 1.0.4 * 1.0.5 * forgot and reset password done * 1.0.6 * Feat/forgot password (#8) * parents translate, components receive plain strings * 1.0.3 * tested in local, bug fixed * 1.0.4 * forgot and reset password done * 1.0.6 * Docs: Add Export Strategy and Workflow Guidelines (#9) * docs: add copilot development instructions - Add React component library development guidelines - Include accessibility and i18n requirements - Document component patterns and testing standards - Add release checklist * docs: create task documentation structure for UI module - Add active tasks folder with UI-MODULE- prefix - Establish workflow for UI component tasks - Add UI-specific documentation focus (accessibility, responsive) * docs: add export strategy and workflow guidelines - Define export strategy (Components + Hooks + Types only) - Document rationale for public API exports - Add workflow with UI-MODULE- prefix convention - Include breaking changes handling for component APIs - Add task documentation structure examples * 1.0.7 * Updated workflows (#10) * ops: updated release pipeline to run only on version changes, and created ci workflow * 1.0.8 * docs(workflow): add Git Flow and npm version requirements (#11) - Add Git Flow branching strategy (develop/master) - Document npm version command before push - Add prepublishOnly hook recommendation - Update workflow with proper branch management - Clear warnings about PR targeting Co-authored-by: Reda Channa <r.channa@ciscod.com> Co-authored-by: Zaiid Moumni <141942826+Zaiidmo@users.noreply.github.com> * docs: added different documentations * 1.0.9 * ops: updated publishing trigger * verify email page done * merged * 1.0.10 * updated endpoints --------- Co-authored-by: Zaiidmo <zaiidmoumnii@gmail.com> Co-authored-by: RedaChannaCiscode <r.channa@ciscod.com> Co-authored-by: Zaiid Moumni <141942826+Zaiidmo@users.noreply.github.com> Co-authored-by: Ciscode-Admin <info@ciscod.com>
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
2 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.
Summary
Why
Checklist
npm run lintpassesnpm run typecheckpassesnpm testpassesnpm run buildpassesnpx changeset) if this affects consumersNotes