Skip to content
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

Old to new onboarding migration fix #3934

Conversation

alessandroboron
Copy link
Contributor

@alessandroboron alessandroboron commented Feb 6, 2025

Task/Issue URL: https://app.asana.com/0/0/1209336232176393/f

Description:

This PR add a fix to set the onboarding Dax Dialogs hidden when the user updates the App if the saw the old onboarding and they haven’t seen some of the dialogs. More info in Asana Task

🎥 Video of the Fix

Old onboarding fresh install:

old_onboarding_to_be_fixed.mp4

Updating the App does not show any dax dialog:

new_onboarding_fixed.mp4

Steps to test this PR:

  1. Remove the DDG App from the simulator.
  2. Checkout branch origin/alessandro/old-onboarding. This is branched off tag 7.130.-2 and include a fix to make it compile with Xcode 16.2.
  3. Launch the App.
  4. Wait for Onboarding to Appear.
  5. Tap the “Let’s Do It” button on the first screen..
  6. When Make DuckDuckGo your default Browser Appear tap the “Skip” button.
  7. When Dax Dialog appear on screen asking to navigate to one of your favourite sites, navigate to duckduckgo.com
  8. When the ’no blocked tracker’ dialog appears on screen, tap the “Got It” button.
  9. Tap the Fire button.
    10.Tap the “Close Tabs and Clear Data” button.
  10. Wait for the “end of journey” (You Got this!) dialogs to appear on screen.
  11. Kill the App.
  12. Switch to origin/alessandro/old-to-new-onboarding-migration-fix branch.
  13. Launch the App.
  14. Ensure that no new dax dialogs are shown in the new tab page.
  15. Navigate to bbc.com
  16. Ensure no dax dialogs are shown.
  17. Navigate to facebook.com
  18. Ensure no dax dialogs are shown.
  19. Navigate to duckduckgo.com
  20. Ensure no dax dialogs are shown.
  21. Clear data using the fire button.
  22. Ensure no end of journey dialog is shown in the new tab page appearing.

Definition of Done (Internal Only):

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPhone 14 Pro
  • iPad

OS Testing:

  • iOS 15
  • iOS 16
  • iOS 17

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template

@alessandroboron alessandroboron changed the title Alessandro/old to new onboarding migration fix Old to new onboarding migration fix Feb 6, 2025
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

LGTM! Nice. Note there's a failing test, but it's fixed on main so don't worry.

I wasn't able to merge because Asana is down and I don't know how this would affect the automations.

@alessandroboron alessandroboron merged commit c589ac2 into release/ios/7.156.0 Feb 6, 2025
24 of 25 checks passed
@alessandroboron alessandroboron deleted the alessandro/old-to-new-onboarding-migration-fix branch February 6, 2025 22:40
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