-
Notifications
You must be signed in to change notification settings - Fork 121
[Just In Time Messages] Add UTM params and analytics to JITM Call to Action #7991
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
[Just In Time Messages] Add UTM params and analytics to JITM Call to Action #7991
Conversation
You can test the changes from this Pull Request by:
|
…tm-params-and-analytics-on-JITM-cta # Conflicts: # WooCommerce/Classes/ViewRelated/Dashboard/DashboardViewModel.swift
…-cta # Conflicts: # WooCommerce/Classes/ViewModels/Feature Announcement Cards/JustInTimeMessageAnnouncementCardViewModel.swift
pachlava
left a comment
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.
Everything worked as you describe @joshheald. I could see the track event with mentioned props, and also the request with the expected params:
I have a question not related to the PR. It looks like Add products to sell banner, that is shown in My store when there are no products, has a precedence over JITM banner. However, once the banner is tapped, and the product had been added, it won't be shown again. I'd expect JITM banner to appear there after screen reload, but it only happens if I switch the store. Is this something known? Thanks!
Screen.Recording.2022-11-01.at.14.48.43.mov
|
Thanks @pachlava!
I hadn't actually tested this yet, but it's not a surprise. Similarly, we don't refresh the JITM except for a full relaunch of the app, logging out and back in, or switching stores, so it could be very out of date in a realistic scenario. This is because we currently do it on I'm going to sync the JITM with the same refresh behaviour as the rest of the screen in a future PR, but I'll also make sure to sync it after the products banner is dismissed. Thanks for pointing it out! |
|
ILGTM! 🎉 |

Closes: #7855
Description
#7973 Added the call to action button to a Just in Time Message, but didn't tackle the analytics for tapping it, nor adding the UTM parameters to the URL if it was for a woocommerce.com link.
This PR resolves those two outstanding items
Testing instructions
N.B. see pdfdoF-1uc-p2#comment-2581 for details of setting up your store to be eligible for the test JITM
UTM params
utm_medium:woo_iosutm_campaign:jitm_group_{FEATURE_CLASS}i.e.jitm_group_woomobile_ipputm_source:my_storeutm_term: Blog IDutm_content:jitm_{JITM_ID}i.e.jitm_woomobile_ipp_barcode_usersWith a US store that is eligible for the test JITM, on a debug/alpha build of the app
Analytics event
*_jitm_cta_tappedeventjitm_group: {FEATURE_CLASS},jitm_id: {JITM_ID},source: my_storeScreenshots
UTM Parameters
Analytics event
RELEASE-NOTES.txtif necessary.