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.

Sign up for GitHub

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

Jump to bottom

Suppress 4xx exceptions when resetting password #279

Merged
merged 3 commits into from
Mar 31, 2020
Merged

Suppress 4xx exceptions when resetting password #279

merged 3 commits into from
Mar 31, 2020

Conversation

DavisRayM
Copy link
Contributor

No description provided.

@DavisRayM DavisRayM force-pushed the handle-4xx-on-password-reset branch from 54f962e to 63bc617 Compare March 30, 2020 12:28
@DavisRayM DavisRayM force-pushed the handle-4xx-on-password-reset branch from 63bc617 to 27292cd Compare March 30, 2020 12:32
ivermac
ivermac approved these changes Mar 30, 2020
View reviewed changes
Copy link
Contributor

@ivermac ivermac left a comment

Choose a reason for hiding this comment

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

looks good to me!

@DavisRayM DavisRayM changed the title Suppress 4xx status codes when resetting password Suppress 4xx exceptions when resetting password Mar 30, 2020
@DavisRayM DavisRayM force-pushed the handle-4xx-on-password-reset branch from 6274b26 to 2bbe0b9 Compare March 31, 2020 09:33
@DavisRayM DavisRayM merged commit 5c96dfb into master Mar 31, 2020
@DavisRayM DavisRayM deleted the handle-4xx-on-password-reset branch March 31, 2020 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ivermac ivermac ivermac approved these changes

@KipSigei KipSigei KipSigei approved these changes

@FrankApiyo FrankApiyo FrankApiyo approved these changes

@RayceeM RayceeM RayceeM approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@DavisRayM @ivermac @KipSigei @FrankApiyo @RayceeM