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

Sucess notification doesn't show when submiting a forgot password form #1777

Closed
vitor-ao opened this issue May 2, 2020 · 6 comments · Fixed by #2562
Closed

Sucess notification doesn't show when submiting a forgot password form #1777

vitor-ao opened this issue May 2, 2020 · 6 comments · Fixed by #2562
Assignees
Labels

Comments

@vitor-ao
Copy link

vitor-ao commented May 2, 2020

Bug Description

After filing the email and submitting to reset the password of a user, the success notification is not rendered on the template, showing the form as it is.

How to Reproduce

The example on https://statamic.dev/tags/user-forgot_password_form can be used, but changing the name="username" to name="email".

Fill the email field with a user in the system and submit. You should receive an email, but no notification is shown on the web page.

Extra Detail

Environment

Statamic version: 3.0.0.-beta-27

PHP version: 7.3.13

Install method (choose one):

  • Fresh install from statamic/statamic
@jackmcdade
Copy link
Member

Were you logged in when you tested that?

@vitor-ao
Copy link
Author

Tested in incognito mode with no user logged in.

@jeremydouglas
Copy link

I'm having a similar experience. I receive this error:

InvalidArgumentException
Password resetter [resets] is not defined. 

@vitor-ao
Copy link
Author

@jeremydouglas for your error check #1973

@mikemartin
Copy link

@vitor-ao See my workaround for this here:
#2086 (comment)

@jesseleite
Copy link
Member

Fixed a bunch of issues around the forgot password form tag, including this one. Should be in next release!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants