feat(auth): password change #226
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR implements a password change feature for authenticated users, adding a new password change form, API integration, validation schema, and UI navigation. While the title mentions "reset," only the password change functionality is implemented in this PR.
Changes:
- Added password change feature with form validation (minimum 8 characters, passwords must match, new password must differ from old)
- Integrated new API endpoint for password changes with error handling
- Added navigation menu item and dedicated page for password changes
- Modified backend logging to use warnings instead of errors for validation failures
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| src/features/password-change/schemas/change-password-schema.ts | Defines validation schema with password matching and difference checks |
| src/features/password-change/index.ts | Feature module exports for components, API, and schemas |
| src/features/password-change/components/change-password-form.tsx | React form component with error handling and toast notifications |
| src/features/password-change/api/change-password.ts | API integration function for password change endpoint |
| src/features/backend/utils/handle-response.ts | Modified to log validation errors as warnings instead of errors |
| src/components/presentation/navbar.tsx | Added "Change Password" menu item to user dropdown |
| src/app/(private)/change-password/page.tsx | New page component for password change functionality |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
kguzek
changed the title
kguzek
previously requested changes
Feb 22, 2026
kguzek
force-pushed
the
feat/password-change-and-reset
branch
from
February 22, 2026 20:02
4d293c6 to
720a346
Compare
kguzek
force-pushed
the
feat/password-change-and-reset
branch
from
March 25, 2026 19:58
720a346 to
e2e1cce
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Member
|
@Konzum59 zrebasuj też brancha:) |
michalges
requested changes
Apr 24, 2026
michalges
left a comment
michalges
left a comment
Member
There was a problem hiding this comment.
Jeśli będziesz robił "Zapomniałem hasła" w osobnej pr'ce to zrób issue od razu
Konzum59
changed the title
michalges
force-pushed
the
feat/password-change-and-reset
branch
from
July 13, 2026 18:25
e588891 to
604af96
Compare
michalges
self-requested a review
July 13, 2026 19:08
michalges
approved these changes
Jul 13, 2026
michalges
dismissed
kguzek’s stale review
July 13, 2026 19:09
naprawione to juz, czemu ten github sie tak pruje
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
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.
No description provided.