Add post launch introduction#22576
Conversation
|
I haven't finished my review yet, but I'm mentioning it now to gain time @pls78 I think the naming has been finalized now and we can use the We do need to have UX give their thumbs up on the copy changes, but we can point them to the RC next week for that |
I think |
leonidasmi
left a comment
There was a problem hiding this comment.
CR + Acceptance is ✅
One thing before merging: I believe that if we are to remove the AI_Brand_Insights_Pre_Launch_Test class we should then add codeCoverageIgnore annotations to its respective AI_Brand_Insights_Pre_Launch class' methods.
I dont think it's blocking a merge, but I think it's worth to mention.
Context
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
Test instructions
Test instructions for the acceptance test before the PR gets merged
This PR can be acceptance tested by following these steps:
wp_usermetatablemeta_key=_yoast_wpseo_introductionsand delete the content of themeta_valuecolumn: this would reset all the introductions you have previously seenOHKjiKm5cSAS7mzI4hK8dd/Introduction?node-id=1-2329&t=MQ5EbyIPwDdC5W5N-4))
Discover Brand Insights nowbutton points tohttps://yoa.st/ai-brand-insights-introduction-post-launch/Relevant test scenarios
Test instructions for QA when the code is in the RC
Impact check
This PR affects the following parts of the plugin, which may require extra testing:
Other environments
[shopify-seo], added test instructions for Shopify and attached theShopifylabel to this PR.Documentation
Quality assurance
Innovation
innovationlabel.Fixes #770