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.

Sign up for GitHub

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

Jump to bottom

Temporary pixel for visit site suggestions CTA display count #5309

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Temporary pixel for visit site suggestions CTA display count #5309

wants to merge 4 commits into from
+37 −0

Conversation

nalcalag
Copy link
Contributor

@nalcalag nalcalag commented Nov 22, 2024 •
edited
Loading

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

Description

Add temporary pixel for counting display of visit site suggestions dialog

Steps to test this PR

Feature 1

  • Fresh install
  • Once in the browser, tap on a search suggestion
  • Dismiss SERP dialog
  • Logcat: Check onboarding_visit_site_cta_shown pixel is fired with parameter count = 0
  • Open a new tab
  • Logcat: Check onboarding_visit_site_cta_shown pixel is fired with parameter count = 1
  • Close and open the app
  • Logcat: Check onboarding_visit_site_cta_shown pixel is fired with parameter count = 2
  • Perform a search
  • Open a new tab
  • Logcat: Check onboarding_visit_site_cta_shown pixel is fired with parameter count = 3
  • Tap on a site suggestion or navigate to a site
  • Check Trackers dialog appears
  • Open a new tab
  • Check End dialog is shown and onboarding_visit_site_cta_shown pixel is not fired

No UI changes

@nalcalag
Copy link
Contributor Author

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

@nalcalag nalcalag force-pushed the feature/noelia/temporary_pixel_for_visit_site_cta_shown_count branch from 4d1cebc to 8c9de35 Compare November 22, 2024 16:25
@nalcalag nalcalag marked this pull request as ready for review November 22, 2024 16:25
@nalcalag nalcalag force-pushed the feature/noelia/temporary_pixel_for_visit_site_cta_shown_count branch from 8c9de35 to 67f34d3 Compare November 22, 2024 16:51
@nalcalag nalcalag force-pushed the feature/noelia/temporary_pixel_for_visit_site_cta_shown_count branch from 1b08248 to d0936d4 Compare November 25, 2024 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cmonfortep cmonfortep

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nalcalag @cmonfortep