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
[Admin][UI] Sending administrator's password reset email #14138
[Admin][UI] Sending administrator's password reset email #14138
Conversation
src/Sylius/Bundle/AdminBundle/Action/Account/PostRequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Resources/config/services/controller.xml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Resources/config/services/controller.xml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Action/Account/GetRequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Action/Account/GetRequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Action/Account/PostRequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Action/Account/PostRequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Action/Account/PostRequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Tests/MessageHandler/Admin/SendResetPasswordEmailHandlerTest.php
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/spec/Action/Account/GetRequestPasswordResetActionSpec.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/spec/Action/Account/PostRequestPasswordResetActionSpec.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Tests/MessageHandler/Admin/SendResetPasswordEmailHandlerTest.php
Show resolved
Hide resolved
@Zales0123 @AdamKasp |
src/Sylius/Bundle/AdminBundle/Action/Account/RenderResetPasswordPageAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Action/Account/RequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Form/Model/PasswordResetRequest.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Form/RequestPasswordResetType.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Action/Account/RequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Action/Account/RequestPasswordResetAction.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/spec/Action/Account/RequestPasswordResetActionSpec.php
Outdated
Show resolved
Hide resolved
@GSadee @NoResponseMate thanks for the suggestions, all introduced 👊🏻. |
src/Sylius/Bundle/AdminBundle/Form/RequestPasswordResetType.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Resources/config/validation/PasswordResetRequest.xml
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I see correctly, you've added validation for both API and UI to this PR, IMO it would be better to add another PR with validation as you've planned before
@GSadee, after a conversation with @NoResponseMate I've dropped his three commits and I've resolved all your comments. While adding information about changes in |
Thank you, Jakub! 🥇 |
…ing a password reset (NoResponseMate) This PR was merged into the 1.12 branch. Discussion ---------- | Q | A | |-----------------|--------------------------------------------------------------| | Branch? | 1.12 | | Bug fix? | yes | | New feature? | no | | BC breaks? | no | | Deprecations? | no | | Related tickets | - | | License | MIT | Based on the work done in #14138 The way a request to reset the password is treated should seemingly be the same regardless if the user exists or not. Commits ------- 720b267 Allow requesting password reset token with a non existent admin email 50140d7 Allow requesting password reset token with a non existent customer email
This PR was merged into the 1.0-dev branch. Discussion ---------- Fixes needed after Sylius/Sylius#14138 1. Allow to access `/admin/forgotten-password` URL anonymously 2. Rework admin login template to use events rather overriding Before: <img width="587" alt="Zrzut ekranu 2022-07-19 o 14 48 02" src="https://user-images.githubusercontent.com/6212718/179754838-3a9c9d36-ad24-404e-b203-8ebce735364c.png"> After: <img width="527" alt="Zrzut ekranu 2022-07-19 o 14 48 16" src="https://user-images.githubusercontent.com/6212718/179754873-9b335198-1769-4e37-abf0-a562b49c3af3.png"> Commits ------- d5ec3d8 Add forgotten password url to anonymous URLs 52248d7 Rework admin login form to events to show "Forgotten password" button
…ResponseMate) This PR was merged into the 1.12 branch. Discussion ---------- | Q | A | |-----------------|--------------------------------------------------------------| | Branch? | 1.12 | | Bug fix? | no | | New feature? | yes | | BC breaks? | no | | Deprecations? | no | | Related tickets | - | | License | MIT | Based on the work done in #14138 Added validation for the admin's request password reset feature. Commits ------- 71cb466 Validation of requesting an admin's password reset token b918b24 [Behat] Extract ResettingPasswordContext out of LoginContext 1d71656 [Behat] Fix methods order in Ui/ResettingPasswordContext
This PR was merged into the 1.0-dev branch. Discussion ---------- Fixes needed after Sylius/Sylius#14138 1. Allow to access `/admin/forgotten-password` URL anonymously 2. Rework admin login template to use events rather overriding Before: <img width="587" alt="Zrzut ekranu 2022-07-19 o 14 48 02" src="https://user-images.githubusercontent.com/6212718/179754838-3a9c9d36-ad24-404e-b203-8ebce735364c.png"> After: <img width="527" alt="Zrzut ekranu 2022-07-19 o 14 48 16" src="https://user-images.githubusercontent.com/6212718/179754873-9b335198-1769-4e37-abf0-a562b49c3af3.png"> Commits ------- d5ec3d88bcecbc2315a0a80ea4c3172ea891aaeb Add forgotten password url to anonymous URLs 52248d7dd7adac1993e59145db891fbb761cde94 Rework admin login form to events to show "Forgotten password" button
Continuation of #14128.
What I've managed to done:
Forgot password?
button on Admin Panel login pageAnd.. flash message 💃🏻.