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

Exclude js.afterpay.com from Minify JS #4526

Merged

Conversation

alfonso100
Copy link
Contributor

@alfonso100 alfonso100 commented Nov 23, 2021

Description

adding js.afterpay.com to the defaults exclusion array in AbstractJSOptimization::get_excluded_external_file_path()

Fixes #4214

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (non-breaking change which improves an existing functionality)

Is the solution different from the one proposed during the grooming?

No

How Has This Been Tested?

on my test site

Checklist:

Please delete the options that are not relevant.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@Tabrisrp Tabrisrp added module: minify JS type: bug Indicates an unexpected problem or unintended behavior labels Nov 24, 2021
@Tabrisrp Tabrisrp requested a review from a team November 24, 2021 20:36
@Tabrisrp Tabrisrp requested a review from a team November 24, 2021 20:37
@piotrbak piotrbak added this to the 3.10.5 milestone Nov 26, 2021
@Mai-Saad Mai-Saad self-requested a review December 1, 2021 11:34
…m-minify-js

# Conflicts:
#	inc/Engine/Optimization/Minify/JS/AbstractJSOptimization.php
Copy link

@Mai-Saad Mai-Saad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working as expected when using afterpay script at the body.
testrail-report-318.pdf

@Mai-Saad Mai-Saad removed the needs: qa label Dec 1, 2021
@iCaspar iCaspar merged commit 19341e4 into develop Dec 1, 2021
@iCaspar iCaspar deleted the fix/4214-exclude-afterpay-external-js-from-minify-js branch December 1, 2021 15:56
@engahmeds3ed engahmeds3ed mentioned this pull request Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: minify JS type: bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Afterpay external script breaks if minified
6 participants