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

Dispatches an event when the user is not found #111

Closed
f-jost opened this issue Dec 12, 2022 · 0 comments · Fixed by #112
Closed

Dispatches an event when the user is not found #111

f-jost opened this issue Dec 12, 2022 · 0 comments · Fixed by #112
Labels
enhancement
Milestone
1.5

Comments

@f-jost
Copy link
Member

f-jost commented Dec 12, 2022 •
edited

Related to #83

Hello @vincentchalamon @GregoireHebert,

What do you think about dispatching an event when the user is not found during the request for the password reset, in order to perform business actions listening to this event?

And updating the status code from 204 to 202?

I propose a pull request about this.
f-jost added a commit to f-jost/CoopTilleulsForgotPasswordBundle that referenced this issue Dec 12, 2022
@f-jost
feat: coopTilleuls#111 dispatch UserNotFoundEvent when a user was not…
6762a4b 
… found
@f-jost f-jost mentioned this issue Dec 12, 2022
Merged
@vincentchalamon vincentchalamon linked a pull request Dec 12, 2022 that will close this issue
feat: #111 dispatch UserNotFoundEvent when a user was not found #112
Merged
@vincentchalamon vincentchalamon added this to the 1.5 milestone Dec 12, 2022
vincentchalamon pushed a commit that referenced this issue Dec 13, 2022
@f-jost
feat: #111 dispatch UserNotFoundEvent when a user was not found (#112)
271c13d 
* feat: #111 dispatch UserNotFoundEvent when a user was not found

* fix: remove friendsofphp/php-cs-fixer dependency

* fix: Update docs/usage.md

* fix: undo ResetPassword status code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
1.5
2 participants
@vincentchalamon @f-jost