Skip to content

Fix Fallback to JQueryUI Compat activated.#95

Closed
frqnck wants to merge 1 commit into
cloudinary:masterfrom
frqnck:patch-2
Closed

Fix Fallback to JQueryUI Compat activated.#95
frqnck wants to merge 1 commit into
cloudinary:masterfrom
frqnck:patch-2

Conversation

@frqnck
Copy link
Copy Markdown

@frqnck frqnck commented May 21, 2021

This short PR fixes:

Fallback to JQueryUI Compat activated. Your store is missing a dependency for a jQueryUI widget. Identifying and addressing the dependency will drastically improve the performance of your site. Compat.js

jquery/ui is deprecated since Magento 2.3 and has been split into dedicated modules to improve performance.

Fix ```Fallback to JQueryUI Compat activated. Your store is missing a dependency for a jQueryUI widget. Identifying and addressing the dependency will drastically improve the performance of your site. Compat.js```
@asisayag2 asisayag2 requested a review from pniel-cohen May 21, 2021 20:41
@frqnck
Copy link
Copy Markdown
Author

frqnck commented Jan 28, 2022

This PR has being incubating here since May 2021?

When will this be merged? Doing so "will drastically improve the performance" of this module on the PDPs (Product Detail Pages) of your customers...

We are patching your code our end but would appreciate not having to have to do this.

@wissam-khalili
Copy link
Copy Markdown

This PR should be merged in our next version.

@wissam-khalili
Copy link
Copy Markdown

Hi @frqnck ,

Have you tried to upgrade to our latest version #112 ?

Regards,
Wissam

@frqnck
Copy link
Copy Markdown
Author

frqnck commented Jun 12, 2023

I can see you have now merged this although you seems to have close this PR and added the change manually so we did not notice the addition until now. We will upgrade accordingly. Thanks you.

@wissam-khalili
Copy link
Copy Markdown

Hi @frqnck ,

I thought I have sent you a notification about the fixed version.
Sorry for the inconvenience.
Please upgrade to the latest version and let me know how it goes,

Regards,
Wissam

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