-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add CMPs #492
base: main
Are you sure you want to change the base?
Add CMPs #492
Conversation
Hi @muodov , Sorry for being late. I missed the notification email. I updated the PR based on the feedback and the latest DOM structures. Please, review the PR again when you have a time. |
"if": { "exists": "div[data-consent-manager-dialog]" }, | ||
"then": [ | ||
{ | ||
"click": "div[data-consent-manager-dialog] form button:not([type='submit'])" |
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.
In my testing, pressing this button was not enough because it triggers another dialog. We should probably actually click the radio buttons in the dialog, rejecting the individual categories, and then click "Save"
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 made some changes by manually referencing the DOM but I wasn't able to get the popup on my side again. Thus, I put another conditional step after all to handle the options popup. If you're able to see it, can you try to test it on your side?
refs ghostery/broken-page-reports#851
refs ghostery/broken-page-reports#847
refs ghostery/broken-page-reports#848
refs ghostery/broken-page-reports#166
refs ghostery/broken-page-reports#203