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.

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

Add a page to change password #261

Merged
merged 3 commits into from
Mar 28, 2023

Conversation

letsintegreat
Copy link
Contributor

Closes #260

Created checkPassword function - passes the username from currentUser and password entered by the user to /authenticate/, returns a boolean value.

Created changePassword function - passes new password and confirm new password fields to /auth/password/change/. If both the passwords are same, and all the required conditions are met, the password is changed. Else, the requirement which is violated is shown in a snackbar.

Created ChangePasswordPage screen.

Demonstration of a few unsuccessful change password attempts with a successful one -

Emulator.-.BLT-Flutter.2023-03-18.02-24-57.mp4

@fredfalcon fredfalcon enabled auto-merge March 17, 2023 21:05
@fredfalcon fredfalcon requested a review from justary27 March 17, 2023 21:05
@fredfalcon
Copy link
Contributor

LGTM!

auto-merge was automatically disabled March 19, 2023 04:16

Head branch was pushed to by a user without write access

@letsintegreat letsintegreat requested a review from justary27 March 19, 2023 04:23
@justary27 justary27 added this pull request to the merge queue Mar 19, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Mar 19, 2023
@justary27 justary27 added this pull request to the merge queue Mar 19, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Mar 19, 2023
@justary27 justary27 added this pull request to the merge queue Mar 24, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Mar 24, 2023
@fredfalcon fredfalcon merged commit 9e8c9c5 into OWASP-BLT:main Mar 28, 2023
DonnieBLT pushed a commit that referenced this pull request Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a page to change the password.
3 participants