-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Marketing Email preview doesn't work with enabled js minification #25068
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 @ihor-sviziev. 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. @ihor-sviziev do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @shikhamis11. Thank you for working on this issue.
|
@magento give me 2.3-develop instance |
Hi @shikhamis11. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @shikhamis11, here is your Magento instance. |
Hello @ihor-sviziev |
@shikhamis11, Just check files contents:
|
Hi @shikhamis11, |
@magento give me 2.3-develop instance |
Hi @ihor-sviziev. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
@magento give me 2.3-develop instance |
Hi @ihor-sviziev. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @ihor-sviziev, here is your Magento instance. |
@shikhamis11 just updated steps to reproduce, we have to have production mode, while on test instance we don't have production mode :( |
Hello @ihor-sviziev you can have production mode in test instance |
@magento give me 2.3-develop instance |
Hi @ihor-sviziev. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
@magento give me 2.3-develop instance |
Hi @ihor-sviziev. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @ihor-sviziev, here is your Magento instance. |
I was able to reproduce this issue on test instance. Just updated issue description with exact steps to reproduce and screenshots of actual result. |
✅ Confirmed by @shikhamis11 Issue Available: @shikhamis11, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
@magento give me 2.3-develop instance |
Hi @LucasCalazans. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @LucasCalazans, here is your Magento instance. |
Hi @arnobsh. Thank you for working on this issue.
|
Internal Magento team is working on this issue right now |
The issue is, that somehow Of course replace versionxxx and en_US to your appropriate paths, versionxxx is only needed when static signing enabled. |
Hi @ihor-sviziev, @arnobsh. Thank you for your report and collaboration! The issue was fixed by Magento team. |
@magento-engcom-team where is the fix? |
Hi @ihor-sviziev, @arnobsh. Thank you for your report and collaboration! The issue was fixed by Magento team. The fix was delivered into The fix will be available with the upcoming |
hello guys, i just faced this issue one 2.3.3 and searched for this ticket and that it hopefully will be fine on 2.3.5. i would only like to note that i did try to copy paste the jsuery.min.js to jquery.js and i get an error: Uncaught SyntaxError: Unexpected end of input for jquery.js the actual file is this one if you would like to have a look. Thank you for your time and effort. |
Hi @ihor-sviziev, @arnobsh. Thank you for your report and collaboration! The issue was fixed by Magento team. The fix was delivered into The fix will be available with the upcoming |
Were these commits deleted? The commit page for both commits is not found |
@linardskrumins now this link available e8eb43a |
Still happening in 2.3.3-p1. The system is looking for A quick (temporary) solution would be to duplicate |
Does not work in 2.4.2 Enterprise in production mode. I get "Blocked script execution in '' because the document's frame is sandboxed and the 'allow-scripts' permission is not set." Is it because I have disabled the module Magento_Csp? EDIT: Solved. It was not because of the javascript. It was caused because the included header could not get loaded, because it was set to a non existing template in "Content -> Design: Configuration -> Theme -> Transaction E-Mails". The problem was caused after copying a database to another project. The error was present in that other project only. |
@Eddcapone, it looks like a bit different issue. Could you report it separately? |
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
The text was updated successfully, but these errors were encountered: