-
Notifications
You must be signed in to change notification settings - Fork 24
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
Feature: Enhanced handling of GDPR banner acceptance in CLI #823
Conversation
Add Texts in a constant and use constant to select a button.
Three items listed were fixed on the latest commit:
|
|
Hi @amovar18, Verified the PR, Observed few things added below:
|
@pavanpatil1 Added a commit for the 3rd feedback, but the first 2 cannot be worked on because in those sites the button is in having an anchor tag with button styling. This breaks the accessibility rules. Our tool finds GDPR banners with acceptance buttons, not anchor tags. |
@ amovar18 anchors are common; can we consider clicking on them? I think block elements like divs and other elements are a no-go. |
I have pulled the latest changes and am still getting a Zero URL counter Command: Selector.json:
|
They are but if the detection is off then it might redirect to another page and mess up the whole analysis. |
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.
Looks good
@amovar18 Please resolve conflicts and we are good to merge. |
…ysis-tool into ref/gdpr-banner-acceptance
@mohdsayed resolved conflicts. |
Description
This PR allows the user to pass a
selector.json
file with flag-b
or--button-selectors
which will be used when any custom selector is used for GDPR banner acceptance.It also adds support for major CMP's like OneTrust, Ibuenda etc.
Relevant Technical Choices
Testing Instructions
npm i && npm run build:cli:dashboard
.npm run cli https://doctolib.fr -- -b selector.json
.Additional Information:
Screenshot/Screencast
Checklist
Fixes #810