-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Improve product edit page loading speed #37477
Improve product edit page loading speed #37477
Conversation
…rver.js to improve the page load performance for product edit pages.
…dom-observer.js to improve the page load performance for product edit pages.
Hi @densen45. Thank you for your contribution! Add the comment under your pull request to deploy test or vanilla Magento instance:
❗ Automated tests can be triggered manually with an appropriate comment:
Allowed build names are:
You can find more information about the builds here For more details, review the Code Contributions documentation. |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@magento run Functional Tests B2B, Functional Tests CE, Functional Tests EE, Static Tests, WebAPI Tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @densen45,
Looks like your changes caused regression according to functional test results. Could you please double-check it?
Hey @ihor-sviziev, Can someone else please check this? |
So for now I'm approving pull request, and someone please help with fixing test failures |
@magento run all tests |
@magento run WebAPI Tests,Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@magento run WebAPI Tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@magento run Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
10d15b0
into
magento:2.4-develop
@densen45 We just applied your fix, and it makes a huge difference (loading time divided by more than 3 in our test case). All our backend team and myself thank you for that ! |
Bad news, the fix actually breaks one of my forms. I'm investigating, and keep you informed. |
@densen45 Got it : With foreach, the |
@Nuranto, can you please create pull request for this change? |
Done. Will ask my team for more tests with this bug fixed. But @densen45 was saying that this one was not the most valuable change of the PR. So I guess it won't change much from our previous results. |
Description (*)
Page loading speed for products with many custom options or bundle products with multiple options with many selections is slow.
The more custom options a product has and the more options and selections a bundle product has, the slower the speed becomes.
In the issue addressed below, several experiences were shared about how slow this can get.
With the changes included in this Pull Request, I was able to reduce the page load time for one of my bundle products (10 options, approx. 160 selections in total) from about 1min 20sec to 35sec.
Related Pull Requests
https://github.com/magento-gl/magento2-page-builder/pull/53
Fixed Issues (if relevant)
Manual testing scenarios (*)
Another scenario would be to add more and more custom options to a product and note that it takes longer to load the edit page.
Questions or comments
Contribution checklist (*)