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

Password reset workflow improvements #2393

Merged
merged 8 commits into from
Dec 18, 2023
Merged

Password reset workflow improvements #2393

merged 8 commits into from
Dec 18, 2023

Conversation

floriankraemer
Copy link
Contributor

PR Description

Password reset workflow improvements.

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md


**src/Pyz/Yves/CustomerPage/CustomerPageConfig.php**

Optional: For security reasons, we recommend to enable the security blocker feature that will block recurring attempts of resetting a password by setting `MERCHANT_PORTAL_SECURITY_BLOCKER_ENABLED` to `true;`
Copy link
Contributor

Choose a reason for hiding this comment

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

A typo: MERCHANT_PORTAL_SECURITY_BLOCKER_ENABLED => CUSTOMER_SECURITY_BLOCKER_ENABLED.


***src/Pyz/Zed/SecurityGui/SecurityGuiConfig.php***

Optional: For security reasons, we recommend to enable the security blocker feature that will block recurring attempts of resetting a password by setting `MERCHANT_PORTAL_SECURITY_BLOCKER_ENABLED` to `true;`
Copy link
Contributor

Choose a reason for hiding this comment

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

  • MERCHANT_PORTAL_SECURITY_BLOCKER_ENABLED => IS_BACKOFFICE_USER_SECURITY_BLOCKER_ENABLED.

@lenadoc lenadoc added the updated PR contains meaningful changes to some document label Dec 18, 2023
@lenadoc lenadoc merged commit 9fec8e6 into spryker:master Dec 18, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech review needed PR needs tech review updated PR contains meaningful changes to some document
Development

Successfully merging this pull request may close these issues.

3 participants