-
Notifications
You must be signed in to change notification settings - Fork 69
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
Updates to Tracks onboarding events props - Take 2 #9036
Updates to Tracks onboarding events props - Take 2 #9036
Conversation
Test the buildOption 1. Jetpack Beta
Option 2. Jurassic Ninja - available for logged-in A12s🚀 Launch a JN site with this branch 🚀 ℹ️ Install this Tampermonkey script to get more options. Build info:
Note: the build is updated when a new commit is pushed to this PR. |
Size Change: +107 B (0%) Total Size: 1.26 MB
ℹ️ View Unchanged
|
Regarding testing instructions:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding the changes, they work well 👏
I tested everything according to the instructions 🎉
The code looks almost good, I left a feedback in the comments, I'll be happy to approve once they're addressed!
Thank you for reviewing this @oaratovskyi !
I've updated the testing instructions.
I've added PHPUnit tests and fixed the type hinting. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for addressing the feedback!
Follow up to #9015
Changes proposed in this Pull Request
source
prop to both Connect page view and Connect CTA clickfrom
param of the requestfrom
param of the requestfrom
param valueTesting instructions
npm run build:client
&from=WCADMIN_PAYMENT_TASK
in the URLwcadmin_page_view
event with thesource: wcadmin-payment-task
proptrunk
zip files and install+activate on your local client installation)&from=WCADMIN_PAYMENT_TASK
in your URL, and that you see a "Connect your store" CTA button.&from=WCADMIN_PAYMENT_TASK
npm run tube:start
and access your local WP admin through it.&from=WCADMIN_PAYMENT_TASK
in the URL&source=wcadmin-payment-task
in the URLnpm run tube:stop
to stop the tunnelingnpm run changelog
to add a changelog file, choosepatch
to leave it empty if the change is not significant. You can add multiple changelog files in one PR by running this command a few times.Post merge