-
Notifications
You must be signed in to change notification settings - Fork 885
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 Wincher upgrade callouts #20239
Add Wincher upgrade callouts #20239
Conversation
@KaisZaoualiWincher Hi, the screenshot of the Dashboard page still shows the combined widget. In Add dashboard wincher connect you split out the keyphrase overview into a separate widget. Has this also been changed in this PR? If so, could you provide an updated screenshot? Thanks! 🙂 |
Hi @manuelaugustin! No, this is not changed in this PR since we didn't base this branch on #20107 (comment). Both branches should not be conflicting since we don't add changes to the widget files in this PR. |
b74ce7d
to
3d2f913
Compare
Hello @vraja-pro! I've pushed a fix for the last issue. It's not related to the current PR, but still, we need to fix it. Thank you for the catch. |
… keyword related actions
0cc4804
to
2f49458
Compare
Context
Add the Wincher account upgrade callouts in both edit-post and dashboard pages.
Summary
This PR can be summarized in the following changelog entry:
Test instructions
Test instructions for the acceptance test before the PR gets merged
Relevant test scenarios
Test instructions for QA when the code is in the RC
QA can test this PR by following these steps:
Impact check
This PR affects the following parts of the plugin, which may require extra testing:
UI changes
Other environments
[shopify-seo]
, added test instructions for Shopify and attached theShopify
label to this PR.Documentation
Quality assurance
Innovation
innovation
label and noted the work hours.Evidence