Skip to content

Adds logic to determine when to show the post signup interstitial#13459

Merged
emilylaguna merged 17 commits intofeature/post-signup-interstitialfrom
issue/12877-add-psi-display-logic
Feb 14, 2020
Merged

Adds logic to determine when to show the post signup interstitial#13459
emilylaguna merged 17 commits intofeature/post-signup-interstitialfrom
issue/12877-add-psi-display-logic

Conversation

@emilylaguna
Copy link
Copy Markdown
Contributor

@emilylaguna emilylaguna commented Feb 13, 2020

Fixes #12877 and #12916

To test:

  1. Launch the app
  2. If you're logged in, log out
  3. Login to an account that has no sites
    3.a Sign up for a new account
  4. You will be shown the epilogue views
  5. Tap next
  6. You should be displayed the interstitial
  7. Relaunch the app, you should not be displayed the interstitial

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.

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Feb 13, 2020

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

Copy link
Copy Markdown
Contributor

@leandroalonso leandroalonso left a comment

Choose a reason for hiding this comment

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

@emilylaguna tested and it works as expected!

I left a few comments, let me know what you think.

@emilylaguna emilylaguna merged commit 41042ea into feature/post-signup-interstitial Feb 14, 2020
@emilylaguna emilylaguna deleted the issue/12877-add-psi-display-logic branch February 14, 2020 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants