You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
All legacy or newly created React components and CSS used for any type of "info" notice components (e.g.<SettingsNotice type='info'> components) other than the above should be removed.
Implementation Brief
The following components should be refactored to directly use the assets/js/components/Notice/index.js component (type = TYPES.INFO) instead of the SettingsNotice component being used within them:
Remove any css classes and duplicated styles for all of the components listed above.
Remove the TYPE_INFO possibility from the assets/js/components/SettingsNotice/SettingsNotice.js component. Remove any associated CSS classes/tests/stories/scenarios for TYPE_INFO of SettingsNotice.
Remove the VARIANTS.INFO possibility from the assets/js/components/notifications/SubtleNotification.js component. Remove any associated CSS classes/tests/stories/scenarios related to this.
Test Coverage
No new tests required.
QA Brief
Notices affected can be found in the PR, but a high level overview of the notices is as follows:
Consent Mode
Plugin compatibility notice informing the user to ensure they have installed a suitable plugin, once WP Consent API has been installed
CM Settings notice in Admin > Settings under Consent Mode that shows when Ads is connected and CM is not connected, ...highly recommended to enable Consent mode...
Adsense
Notice showed near the "Let Site Kit place AdSense code on your site" toggle in AdSense settings
AdBlocker recovery notice: Start recovering revenue lost from ad blockers by deploying an ad blocking recovery message through Site Kit...
AdBlocker recovery notice shown when existing recovery method is detected, You’ve already enabled an ad blocking recovery message on your site...
Reader Revenue Manager
Product ID settings notice: Use the new settings in the block editor to select different product IDs for individual pages or control where CTAs appear on an individual post
Sign In With Google
Notice that shows when "Anyone can register" is disabled: Enable the %1$s setting to allow your visitors to create an account using the %2$s button..., this notices changes slightly depending on whether the Allow new registrations or Anyone can register settings are enabled in WordPress settings.
Tag Manager
Settings notice shown when read only user no longer has access to an account when another admin configured it
General changes
Some warning notices were affected as part of this PR (see files changed > VRT images) due to removal of empty containing elements in the notice when their contents were not present (title wrapper, description wrapper, actions wrapper and padding on description when no actions follow it), but should be able to be QA'd from the VRT changes alone.
Changelog entry
Refactor the usage of all “info type notice” components to use the new <Notice type='info'> component.
Feature Description
This issue will replace all existing / legacy "info" type notifications with the new
<Notice type="info">component created in #10429.InfoNotice

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
<SettingsNotice>components should be refactored to use the new<Notice type="info">component created in Create newNoticeversions of existingSubtleNotifications#10429.<SettingsNotice type='info'>components) other than the above should be removed.Implementation Brief
assets/js/components/Notice/index.jscomponent (type = TYPES.INFO) instead of theSettingsNoticecomponent being used within them:assets/js/components/consent-mode/WPConsentAPIRequirements.jsassets/js/components/settings/SettingsCardConsentMode.jsassets/js/modules/adsense/components/common/UseSnippetSwitch.jsStory 1 | Story 2assets/js/modules/adsense/components/settings/AdBlockingRecoverySetupCTANotice.jsStoryassets/js/modules/adsense/components/settings/AdBlockingRecoveryToggle.jsStory 1 Story 2assets/js/modules/analytics-4/components/settings/AdsConversionIDSettingsNotice.jsassets/js/modules/analytics-4/components/settings/AnalyticsSettingsNotice.jsassets/js/modules/sign-in-with-google/components/common/AnyoneCanRegisterDisabledNotice.jsTYPE_INFOpossibility from theassets/js/components/SettingsNotice/SettingsNotice.jscomponent. Remove any associated CSS classes/tests/stories/scenarios forTYPE_INFOofSettingsNotice.VARIANTS.INFOpossibility from theassets/js/components/notifications/SubtleNotification.jscomponent. Remove any associated CSS classes/tests/stories/scenarios related to this.Test Coverage
QA Brief
Notices affected can be found in the PR, but a high level overview of the notices is as follows:
Consent Mode
...highly recommended to enable Consent mode...Adsense
Start recovering revenue lost from ad blockers by deploying an ad blocking recovery message through Site Kit...You’ve already enabled an ad blocking recovery message on your site...Reader Revenue Manager
Use the new settings in the block editor to select different product IDs for individual pages or control where CTAs appear on an individual postSign In With Google
Enable the %1$s setting to allow your visitors to create an account using the %2$s button..., this notices changes slightly depending on whether theAllow new registrationsorAnyone can registersettings are enabled in WordPress settings.Tag Manager
General changes
Changelog entry
<Notice type='info'>component.