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
CYS: fix PHP warnings and pattern button #43354
Conversation
Test Results SummaryCommit SHA: 7965529
To view the full API test report, click here. To view the full E2E test report, click here. To view all test reports, visit the WooCommerce Test Reports Dashboard. |
Apart from reviewing the code changes, please make sure to review the testing instructions as well. You can follow this guide to find out what good testing instructions should look like: |
b68e5ca
to
d0c7714
Compare
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.
Great work @albarin !
Insert the Banner pattern, save, and make sure the button does not have custom styles (see screenshots below).
✅ Confirmed the button doesn't have custom styles
Check the debug.log and make sure there are no PHP warnings related to slug key not being present and $images not being defined.
✅ Confirmed that there are no PHP warnings in the error log.
* Remove custom styles from button * Avoid php warning about images and slug not defined * Add changefile(s) from automation for the following project(s): woocommerce * Add changefile(s) from automation for the following project(s): woocommerce * Fix linting errors --------- Co-authored-by: github-actions <github-actions@github.com>
* CYS: fix PHP warnings and pattern button (#43354) * Remove custom styles from button * Avoid php warning about images and slug not defined * Add changefile(s) from automation for the following project(s): woocommerce * Add changefile(s) from automation for the following project(s): woocommerce * Fix linting errors --------- Co-authored-by: github-actions <github-actions@github.com> * Prep for cherry pick 43354 --------- Co-authored-by: Alba Rincón <albarin@users.noreply.github.com> Co-authored-by: github-actions <github-actions@github.com> Co-authored-by: WooCommerce Bot <no-reply@woo.com>
Submission Review Guidelines:
Changes proposed in this Pull Request:
slug
key not present and$images
variable not being defined.Banner
pattern button.How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
Customize your store
flow, create a new page or post.Banner
pattern, save, and make sure the button does not have custom styles (see screenshots below).debug.log
and make sure there are no PHP warnings related toslug
key not being present and$images
not being defined.Changelog entry
Significance
Type
Message
Fix PHP warnings and remove opinionated styles from the
Banner
button.Comment