-
Notifications
You must be signed in to change notification settings - Fork 683
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
BUG#3982 Add uid instead of id in noProductsFound.js #3982
BUG#3982 Add uid instead of id in noProductsFound.js #3982
Conversation
|
Hello @Jordaneisenburger , Can you please check this PR that have only one check left. |
Hello @anthoula, Can you please check this too? |
run cypress |
Successfully started codebuild job for |
run pr-test |
Successfully started codebuild job for |
LogERROR ON TASK: cypressTests
|
QA Aprroved |
Description
We have already deprecated id field and we will replace it with uid changes but there is still missing one place in category page if there is no products for category then below component is called that used category.id instead of category.uid. Id is not there in object that gives console warning for key.
So we have updated from id to uid that gives unique value for each category and resolved the browser console warning.
Related Issue
Closes #3981.
Acceptance
Verification Stakeholders
Specification
Verification Steps
Test scenario(s) for direct fix/feature
Test scenario(s) for any existing impacted features/areas
Test scenario(s) for any Magento Backend Supported Configurations
Is Browser/Device testing needed?
Any ad-hoc/edge case scenarios that need to be considered?
Screenshots / Screen Captures (if appropriate)
Breaking Changes (if any)
Checklist