-
Notifications
You must be signed in to change notification settings - Fork 295
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
Include 'Learn more' link on RRM set up screen #9259
Comments
I've raised a question about this issue on its Asana task, and will hold off adding AC while we consider it further. |
Hi @wpdarren @aaemnnosttv, I'm assigning this to you for visibility, please can you take a look at the linked Asana task when you have a moment? |
@techanvil I've replied on that, thanks for raising it. |
Thanks @aaemnnosttv! I've added AC for this issue, including a more contextually relevant link as per your suggestion on Asana. |
AC ✅ |
IB ✅ |
QA Update ✅This was verified good. Moving ticket to approval.
|
Feature Description
On the Reader Revenue Manager setup screens it would be nice to include a
Learn more
link so the user can find out more about the feature before setting it up.Asana ticket can be found here
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation Brief
assets/js/modules/reader-revenue-manager/components/common/PublicationCreate.js
, usecreateInterpolateElement
to add a Learn More link, linking to the support article in the AC.site-kit-wp/assets/js/modules/reader-revenue-manager/components/common/PublicationCreate.js
Lines 80 to 85 in c762066
Test Coverage
QA Brief
Learn More
link pointing to https://support.google.com/news/publisher-center/answer/11449914Changelog entry
The text was updated successfully, but these errors were encountered: