-
Notifications
You must be signed in to change notification settings - Fork 1.1k
SERP Settings Sync: Add and update pixels #7073
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
SERP Settings Sync: Add and update pixels #7073
Conversation
f3afcfb to
6e5ebf2
Compare
| DUCK_CHAT_EXPERIMENTAL_ADDRESS_BAR_IS_ENABLED_DAILY.pixelName to PixelParameter.removeAtb(), | ||
| DUCK_CHAT_SEARCH_ASSIST_SETTINGS_BUTTON_CLICKED.pixelName to PixelParameter.removeAtb(), | ||
| DUCK_CHAT_HIDE_AI_GENERATED_IMAGES_BUTTON_CLICKED.pixelName to PixelParameter.removeAtb(), | ||
| SERP_SETTINGS_OPEN_HIDE_AI_GENERATED_IMAGES.pixelName to PixelParameter.removeAtb(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to move this to the settings module even though it's only fired here?
If we do then we'll need to make an API proposal to expose it.
I'm not sure it's worth it, we only renamed it to match other platforms.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think is fine leaving it here
serp_settings_open_hide-ai-generated-images is going to replace an existing pixel name serp_settings_open_duck-ai is new
Add our own plugin so we're not using app's PixelInterceptor
6e5ebf2 to
824bbce
Compare
|
Privacy Review task: https://app.asana.com/0/69071770703008/1211991948393724 |
This stack of pull requests is managed by Graphite. Learn more about stacking. |
marcosholgado
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM

Task/Issue URL: https://app.asana.com/1/137249556945/project/1207908166761516/task/1211846254913290?focus=true
Description
Added and updated pixels for SERP settings sync work to match other platforms.
serp_settings_open_hide-ai-generated-imagesis going to replace an existing pixel nameHide Ai Generatedoption is opened and highlighted in SERP settings.serp_settings_open_duck-aiis newSteps to test this PR
serp_settings_open_hide-ai-generated-imagespixel is fired and no ATBserp_settings_open_duck-aipixel is fired with no ATB and you're taken to the AI Features settings page