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

Web pixels (handlers + pixels) #2611

Merged
merged 3 commits into from
Mar 19, 2024

Conversation

federicocappelli
Copy link
Member

@federicocappelli federicocappelli commented Mar 18, 2024

Task/Issue URL: https://app.asana.com/0/72649045549333/1205469290776415/f
Implementation sub-task: https://app.asana.com/0/72649045549333/1206697233126329/f
CC: @miasma13

Description:

The FE needs to fire pixels after some user actions, this PR implements the web handlers where the related pixels are fired.

Steps to test this PR:

  1. 4 pixels are implemented (see https://app.asana.com/0/72649045549333/1206697233126329/f)
  2. For each Pixel recreate the action described in the Trigger and see if the right pixel is fired.

@federicocappelli federicocappelli changed the title handlers for Web pixels implemented Web pixels (handlers + pixels) Mar 18, 2024
@github-actions github-actions bot added the bot: not in app board Added by automation for pull requests with tasks not added to iOS App Board Asana project label Mar 18, 2024
@github-actions github-actions bot removed the bot: not in app board Added by automation for pull requests with tasks not added to iOS App Board Asana project label Mar 18, 2024
@duckduckgo duckduckgo deleted a comment from github-actions bot Mar 18, 2024
Copy link
Contributor

@miasma13 miasma13 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@federicocappelli federicocappelli merged commit b1b2286 into main Mar 19, 2024
15 of 16 checks passed
@federicocappelli federicocappelli deleted the fcappelli/subscription_web_pixels branch March 19, 2024 10:56
samsymons added a commit that referenced this pull request Mar 20, 2024
# By Anh Do (2) and others
# Via GitHub
* main:
  23. Subscriptions:  - Feature Flags (#2605)
  fix settings maestro text (#2613)
  Web pixels (handlers + pixels) (#2611)
  Handle subscription-related iOS use cases (#2597)
  Remove hardcoded NetP staging endpoint (#2612)
  Add Privacy Manifests to app extensions (#2608)
  Add a VPN metadata section to the debug UI (#2502)
  Use History in Suggestions on iOS (#2552)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
samsymons added a commit that referenced this pull request Mar 21, 2024
# By Sam Symons (3) and others
# Via GitHub
* main:
  Entitlements cache fixes (#2620)
  BSK update for RMF app build version improvement (#2564)
  Surface the last-used password first in autofill prompt when filling passwords (#2600)
  VPN UI improvements part 1 - basic copy and UI changes (#2580)
  Add setting to display full URL address (#2587)
  Don't refresh swipe tabs model on background changes (#2610)
  23. Subscriptions:  - Feature Flags (#2605)
  fix settings maestro text (#2613)
  Web pixels (handlers + pixels) (#2611)
  Handle subscription-related iOS use cases (#2597)
  Remove hardcoded NetP staging endpoint (#2612)
  Add Privacy Manifests to app extensions (#2608)
  Add a VPN metadata section to the debug UI (#2502)
  Use History in Suggestions on iOS (#2552)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
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