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

DBP: Send internal user param for dbp waitlist pixels #1972

Merged
merged 4 commits into from
Dec 15, 2023

Conversation

jotaemepereira
Copy link
Collaborator

Task

https://app.asana.com/0/1204006570077678/1206166258660238/f

Description

Adds the isInternalUser flag to the DBP waitlist pixels

Steps to test

  1. Log in as internal user
  2. Reset the waitlist of DBP from the debug menu
  3. Check that the pixels that are being sent have the isInternalUser flag to true

Do the same but without being an internal user.

Copy link
Collaborator

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

LGTM. Did you want to also fix the region check on the active user pixel here as well? I think it makes sense to bundle these two things into one PR.

@jotaemepereira jotaemepereira force-pushed the juan/dbp/fire-waitlist-pixels-if-internal branch from 88c638c to 1f9868d Compare December 15, 2023 13:03
@jotaemepereira jotaemepereira merged commit d45bc8f into main Dec 15, 2023
14 checks passed
@jotaemepereira jotaemepereira deleted the juan/dbp/fire-waitlist-pixels-if-internal branch December 15, 2023 13:29
samsymons added a commit that referenced this pull request Dec 15, 2023
* main:
  Updates to Autofill Logins copy (#1924)
  Bump version to 1.69.0 (95)
  Remove the reconnect/disconnect logic from the connection tester (#1970)
  Set marketing version to 1.69.0
  Update embedded files
  DBP: Send internal user param for dbp waitlist pixels (#1972)
  Move release task to proper section in Code Freeze workflow (#1977)
  drop Main.storyboard (#1944)
  Add GHA workflow to cut release branch (#1976)
  Move DBP tests into main target (#1974)
  Use static date for PixelKit tests (#1973)
samsymons added a commit that referenced this pull request Dec 15, 2023
* main: (48 commits)
  Add additional VPN startup pixels (#1975)
  Updates to Autofill Logins copy (#1924)
  Bump version to 1.69.0 (95)
  Remove the reconnect/disconnect logic from the connection tester (#1970)
  Set marketing version to 1.69.0
  Update embedded files
  DBP: Send internal user param for dbp waitlist pixels (#1972)
  Move release task to proper section in Code Freeze workflow (#1977)
  drop Main.storyboard (#1944)
  Add GHA workflow to cut release branch (#1976)
  Move DBP tests into main target (#1974)
  Use static date for PixelKit tests (#1973)
  Remove DBP test target (#1961)
  Fix date generator for time machine (#1969)
  Improve sync set up error handling (#1966)
  remove QR code from save recovery PDF view (#1968)
  change order of items in autofill add new item (#1967)
  Fix PR Checks workflow (#1962)
  Bump version to 1.68.0 (93)
  Update embedded files
  ...
samsymons added a commit that referenced this pull request Dec 19, 2023
* main:
  DBP: Fix unreliable date tests (#1981)
  Add search retention pixel for NetP (#1964)
  Sabrina/sync e2e tests (#1959)
  swiftlint build plugin (#1318)
  VPN Geoswitching - initial draft (#1978)
  Add additional VPN startup pixels (#1975)
  Updates to Autofill Logins copy (#1924)
  Bump version to 1.69.0 (95)
  Remove the reconnect/disconnect logic from the connection tester (#1970)
  Set marketing version to 1.69.0
  Update embedded files
  DBP: Send internal user param for dbp waitlist pixels (#1972)
  Move release task to proper section in Code Freeze workflow (#1977)
  drop Main.storyboard (#1944)
  Add GHA workflow to cut release branch (#1976)
  Move DBP tests into main target (#1974)
  Use static date for PixelKit tests (#1973)
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