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

Feature: Allow donors to update or reset their password #7125

Merged
merged 3 commits into from
Jan 9, 2024

Conversation

kjohnson
Copy link
Member

@kjohnson kjohnson commented Dec 5, 2023

Description

This PR adds a Password Reset feature to the Donor Dashboard login modal.

Visuals

Password Reset Button

Screenshot 2023-12-05 at 16-29-09 Donor Dashboard – WordPress

Password Reset Form

Screenshot 2023-12-05 at 16-29-16 Donor Dashboard – WordPress

Password Reset Sent

Screenshot 2023-12-05 at 16-29-26 Donor Dashboard – WordPress

@kjohnson kjohnson marked this pull request as ready for review December 5, 2023 23:19
@JoshuaHungDinh
Copy link
Contributor

I tested this PR out. Everything works and looks good! Just left a comment on the unreleased tags in the password route.

@kjohnson
Copy link
Member Author

kjohnson commented Dec 7, 2023

Just left a comment on the unreleased tags in the password route.

Yup. Copy/paste strikes again!

Resolved, see abc4ba0

src/DonorDashboards/Routes/PasswordResetRoute.php Outdated Show resolved Hide resolved
src/DonorDashboards/Routes/PasswordResetRoute.php Outdated Show resolved Hide resolved
@kjohnson
Copy link
Member Author

Copy link
Member

@rickalday rickalday left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Passed manual QA tests

@jonwaldstein jonwaldstein merged commit 6022f92 into develop Jan 9, 2024
20 checks passed
@jonwaldstein jonwaldstein deleted the GIVE-29 branch January 9, 2024 20:59
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.

4 participants