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

Confirmation emails have no FROM or FROM email address 2.2.4 #14952. … #15087

Closed
wants to merge 3 commits into from

Conversation

sashas777
Copy link
Contributor

@sashas777 sashas777 commented May 8, 2018

Undone commit f2bfdd9 which caused issue

Description

Removed class lib/internal/Magento/Framework/Mail/Template/TransportBuilderByStore and assosiated tests for it.
The class cause the issue #14952 and was introduced because in previous releases default class lib/internal/Magento/Framework/Mail/Template/TransportBuilder didn't have the parameter $store for function public function setFromByStore($from, $store) Line 291.

Fixed Issues (if relevant)

  1. Confirmation emails have no FROM or FROM email address 2.2.4 #14952: Confirmation emails have no FROM or FROM email address 2.2.4
  2. Remove var Translator = new Translate... from every page #24: Wrong sender name/email in invoice/shipment email in multistore

Manual testing scenarios

  1. When Magento has single store view: set values at the admin Store->Configuration->General->Store Email Addresses-> Sales Representative. Make an order from storefront then check in the order success email "name from" and "email from" fields. It supposes to be matched to the admin configuration.

  2. When Magento has single store view: set values at the admin Store->Configuration->General->Store Email Addresses-> Sales Representative. Make an order from admin panel then check in the order success email "name from" and "email from" fields. It supposes to be matched to the admin configuration.

  3. When Magento has multiple store views: set different values at the admin Store->Configuration->General->Store Email Addresses-> Sales Representative.

  4. Create a new order from the storefront for each store view and look at your email to confirm these values in order confirmation email.

  5. Create a new order from the admin for each store view and look at your email to confirm these values in order confirmation email.

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-cicd2
Copy link
Contributor

magento-cicd2 commented May 8, 2018

CLA assistant check
All committers have signed the CLA.

@sashas777 sashas777 closed this May 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants