Skip to content

Adds analytics for the Post Signup Interstitial#13505

Merged
emilylaguna merged 1 commit intofeature/post-signup-interstitialfrom
issue/12880-add-welcome-interstitial-analytics
Feb 19, 2020
Merged

Adds analytics for the Post Signup Interstitial#13505
emilylaguna merged 1 commit intofeature/post-signup-interstitialfrom
issue/12880-add-welcome-interstitial-analytics

Conversation

@emilylaguna
Copy link
Copy Markdown
Contributor

@emilylaguna emilylaguna commented Feb 19, 2020

Fixes #12880
Shared PR: wordpress-mobile/WordPress-iOS-Shared#249

To test:

  1. Launch the app

  2. If you're logged in, log out

  3. Tap Continue with WordPress.com

  4. Login to an account that has no sites

  5. When the PSI is displayed, verify this in the console:
    🔵 Tracked: welcome_no_sites_interstitial_shown

  6. Tap the create site button, and verify this in the console:
    🔵 Tracked: welcome_no_sites_interstitial_button_tapped <button: create_new_site>

  7. Tap the add self-hosted site button, and verify this in the console:
    🔵 Tracked: welcome_no_sites_interstitial_button_tapped <button: add_self_hosted_site>

  8. Tap the Not Right Now option, and verify this in the console:
    🔵 Tracked: welcome_no_sites_interstitial_dismissed

Note: In order to test each button option you will need to either:

  • Delete and relaunch the app after each button press since the PSI will not be displayed again for your account.
  • Login using a different account for each action

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@emilylaguna emilylaguna added this to the 14.3 milestone Feb 19, 2020
@emilylaguna emilylaguna self-assigned this Feb 19, 2020
@peril-wordpress-mobile
Copy link
Copy Markdown

You can trigger an installable build for these changes by visiting CircleCI here.

Copy link
Copy Markdown
Contributor

@ScoutHarris ScoutHarris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ When the PSI is displayed
✅ Tap the create site button
✅ Tap the add self-hosted site button
✅ Tap the Not Right Now option

LGTM!

:shipit:

@emilylaguna emilylaguna removed the request for review from diegoreymendez February 19, 2020 20:12
@emilylaguna emilylaguna merged commit a064292 into feature/post-signup-interstitial Feb 19, 2020
@emilylaguna emilylaguna deleted the issue/12880-add-welcome-interstitial-analytics branch February 19, 2020 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants