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

[Forwardport] Fix reset password link with appropriate customer store #15095

Merged

Conversation

rogyar
Copy link
Contributor

@rogyar rogyar commented May 9, 2018

Original PR: #14800

Fix issue #5726 to send the reset password message from backend using the store defined in customer

Description

Change the function passwordReminder to use the storeId defined in customer. As a fallback if no store is defined in customer, will be used the default store of customer website.

Fixed Issues (if relevant)

  1. Reset Password Email Issue on Multi Store from Admin #5726: Reset Password Email Issue on Multi Store from Admin

Manual testing scenarios

  1. Create multiple stores
  2. Define different languages for each on
  3. Open the customer in admin page
  4. Click on reset password button
  5. Change the store in Send Welcome Email From
  6. Save the customer
  7. Click on reset password button

@magento-engcom-team magento-engcom-team added this to the May 2018 milestone May 9, 2018
@magento-engcom-team magento-engcom-team added Partner: Atwix Pull Request is created by partner Atwix partners-contribution Pull Request is created by Magento Partner labels May 9, 2018
@magento-engcom-team
Copy link
Contributor

Hi @rogyar. Thank you for your contribution.
Changes from your Pull Request will be available with the upcoming 2.3.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Partner: Atwix Pull Request is created by partner Atwix partners-contribution Pull Request is created by Magento Partner Progress: accept Release Line: 2.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants