-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Line length is too long in emails #24531
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
Comments
Hi @philippeaellig. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @philippeaellig do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @edenduong. Thank you for working on this issue.
|
Hi @engcom-Bravo. Thank you for working on this issue.
|
Fixed in 2.2-develop. Will be in 2.2.10 release |
See PR #23650 |
Uh oh!
There was an error while loading. Please reload this page.
Preconditions (*)
Magento 2.2.8 in production mode. Use production mode to reproduce this bug, because only in this case, less code will compiled and minimized to one line.
Steps to reproduce (*)
Send an email with the default magento header.
vendor/magento/module-email/view/frontend/email/header.html
Have a look at the source code of the mail.
The line with the rendered css from css/email.css is too long, in my case 5611 octets. this is too long. allowed are max. 998 characters for one line. some mails will not reach the reciver, because some mailservers do not accept mails with long lines.
Expected result (*)
Actual result (*)
I think, this issue is about the same topic. https://github.com/mageplaza/magento-2-smtp/issues/156 (another magento extension, but this is not a mageplaza bug...)
The text was updated successfully, but these errors were encountered: