[Step 2] mu-wpcom: remove the COLOURLovers Customizer deprecation notice#50012
Conversation
COLOURLovers support has been discontinued and the deprecation deadline has passed, so the Customizer warning that asked users to update their background image is now stale. PR1 already blanks COLOURLovers backgrounds at render. Remove the wpcom-colourlovers-deprecate feature (PHP + JS) and its loader in class-jetpack-mu-wpcom.php.
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 Follow this PR Review Process:
If you have questions about anything, reach out in #jetpack-developers for guidance! |
There was a problem hiding this comment.
Pull request overview
Removes the now-stale COLOURLovers background deprecation notice feature from jetpack-mu-wpcom, including its PHP/JS implementation and loader hook, and records the removal in the project changelog. This aligns with COLOURLovers support being discontinued and the deadline having passed (with related behavior already handled in stacked PR #50011).
Changes:
- Delete the
wpcom-colourlovers-deprecatefeature PHP and its Customizer warning script. - Stop loading the removed feature from
Jetpack_Mu_Wpcom::load_wpcom_user_features(). - Add a
jetpack-mu-wpcomchangelog entry documenting the removal.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| projects/packages/jetpack-mu-wpcom/src/features/wpcom-colourlovers-deprecate/wpcom-colourlovers-deprecate.php | Removes the PHP hook that enqueued/localized the Customizer deprecation warning. |
| projects/packages/jetpack-mu-wpcom/src/features/wpcom-colourlovers-deprecate/js/colourlovers-deprecated-message.js | Removes the Customizer UI notification logic for the deprecated COLOURLovers background warning. |
| projects/packages/jetpack-mu-wpcom/src/class-jetpack-mu-wpcom.php | Removes the require_once loader line for the deleted feature. |
| projects/packages/jetpack-mu-wpcom/changelog/dotcom-1758-remove-colourlovers-deprecate-notice | Adds a changelog entry documenting the removal of the stale Customizer notice. |
Proposed changes
wpcom-colourlovers-deprecatefeature — the Customizer notice that asked users to update their COLOURLovers background image before the deprecation deadline. COLOURLovers support has been discontinued and the deadline has passed, so the warning is stale. [Step 1] Custom colors: stop serving discontinued COLOURLovers backgrounds #50011 already blanks COLOURLovers backgrounds at render time.class-jetpack-mu-wpcom.php.Related product discussion/links
Does this pull request change what data or activity we track or use?
No.
Testing instructions
jetpack-mu-wpcomand deploy to an Atomic site.function_exists( 'wpcom_colourlovers_deprecate_warning' )isfalse.