Skip to content

Conversation

@rodrigowebjump
Copy link
Member

@rodrigowebjump rodrigowebjump commented Apr 22, 2018

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

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@magento-engcom-team
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants