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

Update pixels in onboarding experiment #4584

Merged
merged 1 commit into from
May 27, 2024

Conversation

nalcalag
Copy link
Contributor

@nalcalag nalcalag commented May 24, 2024

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

Description

Add three unique pixels when pre-onboarding dialogs are shown

Steps to test this PR

Pre steps

  • Make fun isComparisonChartEnabled(): Boolean return always true in ExtendedOnboardingExperimentVariantManagerImpl

Feature 1

  • Fresh install
  • When first dialog is displayed, check m_preonboarding_intro_shown_unique is fired
  • When comparison chart is displayed, check m_preonboarding_comparison_chart_shown_unique is fired
  • Set DDG as default browser
  • When affirmation dialog is displayed, check m_preonboarding_affirmation_shown_unique is fired

No UI changes

@nalcalag
Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nalcalag and the rest of your teammates on Graphite Graphite

@nalcalag nalcalag marked this pull request as ready for review May 27, 2024 08:55
Copy link
Contributor

@CrisBarreiro CrisBarreiro left a comment

Choose a reason for hiding this comment

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

Looks good and works as expected

@nalcalag nalcalag merged commit 4232a82 into develop May 27, 2024
7 checks passed
@nalcalag nalcalag deleted the feature/noelia/unique_pixels_pre_onboarding branch May 27, 2024 10:29
cmonfortep pushed a commit that referenced this pull request May 27, 2024
Task/Issue URL:
https://app.asana.com/0/1201807753394693/1207407312592951/f

### Description
Add three unique pixels when pre-onboarding dialogs are shown

### Steps to test this PR
_Pre steps_

- [x] Make `fun isComparisonChartEnabled(): Boolean` return always
`true` in `ExtendedOnboardingExperimentVariantManagerImpl`

_Feature 1_
- [x] Fresh install
- [x] When first dialog is displayed, check
`m_preonboarding_intro_shown_unique` is fired
- [x] When comparison chart is displayed, check
`m_preonboarding_comparison_chart_shown_unique` is fired
- [x] Set DDG as default browser
- [x] When affirmation dialog is displayed, check
`m_preonboarding_affirmation_shown_unique` is fired

### No UI changes
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.

None yet

2 participants