-
Notifications
You must be signed in to change notification settings - Fork 874
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
Add upsell for Local SEO to search appearance #12646
Add upsell for Local SEO to search appearance #12646
Conversation
This outputs the difference in style of styled-components to the snapshots. Making the right-to-left style difference testable with snapshots. Update existing snapshots to match.
…psell-for-local-seo-to-search-appearance
While looking at Yoast/javascript#220 I see that the adding of |
CR 👍 |
…moved a11y attributes
I'm not getting this reproduced, but I did get a nice plugin from Alexander to be able to test this easier! Install test pluginSee https://github.com/Yoast/Wiki/wiki/Filter-the-subscriptions-response-in-the-addon-manager-request |
@IreneStr I cannot reproduce your situation either. But should we look into it again before merging this PR? |
It looks like it was cache-related, so it can be merged after your thumbs up @johannadevos |
Acceptance 👍 |
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
react-svg-loader
, which turns it into a component.wpseo-local-seo-upsell
). Then the component is mounted on that depending on whether we want the upsell there or not. I left this because adding more logic in the PHP view is not really nice either. If you have a strong opinion about this, please share.Test instructions
This PR can be tested by following these steps:
Viewing the upsell.
Organization
as representation under theKnowledge Graph
setting.Install the helper plugin to manipulate the license
SEO -> Premium
page (see Irene's screenshot). Which is also where you can see the current status.The upsell should only show when you do not have an active Local SEO license.
This is a bit tricky as it depends on which licenses are active on your site.You can create another vagrant box to test a site without a license.Or you can edit the PHP code. Inadmin/class-config.php
on line 143 change the value ofshowLocalSEOUpsell
intotrue
orfalse
.expiryDate
in the helper plugin to a date in the past (2019-01-27T01:00:00.000Z
).SEO -> Premium
page to see if the status of Local SEO is indeedNOT ACTIVATED
.expiryDate
in the helper plugin to a date in the future (2019-12-27T01:00:00.000Z
).SEO -> Premium
page to see if the status of Local SEO is indeedACTIVATED
.The upsell should only show when you are in Free, never in Premium.
There is RTL support.
The link should contain UTM tags and other info
https://yoast.com/wordpress/plugins/local-seo/#utm_source=yoast-seo&utm_medium=software&utm_term=local-seo-kb-notification&utm_content=settings&utm_campaign=wordpress-ad
php_version
andplatform_version
.Screenshot
UI changes
Quality assurance
Fixes https://github.com/Yoast/wordpress-seo-premium/issues/2127