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

Added exception for larazon.es #2020

Merged
merged 1 commit into from
Apr 23, 2024
Merged

Added exception for larazon.es #2020

merged 1 commit into from
Apr 23, 2024

Conversation

tagawa
Copy link
Contributor

@tagawa tagawa commented Apr 23, 2024

Asana Task/Github Issue:

Description

Steps to repro:
Go to https://www.larazon.es/
Wait or scroll around a bit until you see the cookie popup.

What should happen:
You should see a “Rechazar” (reject) button which hides the cookie popup when clicked, or ideally it should disappear automatically with our cookie popup management.

What actually happens:
You get a list of cookies to refuse. If you click the "Rechazar todo” (reject all) button they are rejected but the cookie popup doesn’t disappear. See attached “normal” and “unusable” screenshots.

It looks like our cookie popup management is clicking the “Configurar” (configure) button on the initial popup instead of “Rechazar” (reject).

Works fine in other browsers or with “Cookie Pop-up Protection” disabled.

Reference

@tagawa tagawa requested review from muodov, sammacbeth and a team as code owners April 23, 2024 07:34
@tagawa tagawa added the merge when ready If you approve this PR, merge it immediately label Apr 23, 2024
Copy link

Generated file outputs:

legacy
fingerprinting.json (3 more)
  • fingerprinting.json
  • protections.json
  • trackers-unprotected-temporary.txt
  • trackers-whitelist-temporary.txt

⚠️ File is identical

v1/android-config.json (41 more)
  • v1/android-config.json
  • v1/extension-brave-config.json
  • v1/extension-bravemv3-config.json
  • v1/extension-chrome-config.json
  • v1/extension-chromemv3-config.json
  • v1/extension-config.json
  • v1/extension-edg-config.json
  • v1/extension-edge-config.json
  • v1/extension-edgmv3-config.json
  • v1/extension-firefox-config.json
  • v1/extension-safarimv3-config.json
  • v1/ios-config.json
  • v1/macos-config.json
  • v1/windows-config.json
  • v2/android-config.json
  • v2/extension-brave-config.json
  • v2/extension-bravemv3-config.json
  • v2/extension-chrome-config.json
  • v2/extension-chromemv3-config.json
  • v2/extension-config.json
  • v2/extension-edg-config.json
  • v2/extension-edge-config.json
  • v2/extension-edgmv3-config.json
  • v2/extension-firefox-config.json
  • v2/extension-safarimv3-config.json
  • v2/ios-config.json
  • v2/macos-config.json
  • v2/windows-config.json
  • v3/android-config.json
  • v3/extension-brave-config.json
  • v3/extension-bravemv3-config.json
  • v3/extension-chrome-config.json
  • v3/extension-chromemv3-config.json
  • v3/extension-config.json
  • v3/extension-edg-config.json
  • v3/extension-edge-config.json
  • v3/extension-edgmv3-config.json
  • v3/extension-firefox-config.json
  • v3/extension-safarimv3-config.json
  • v3/ios-config.json
  • v3/macos-config.json
  • v3/windows-config.json
--- v3/windows-config.json
+++ v3/windows-config.json
@@ -144,8 +144,12 @@
                     "domain": "ksta.de",
                     "reason": "https://github.com/duckduckgo/privacy-configuration/issues/326"
                 },
                 {
+                    "domain": "larazon.es",
+                    "reason": "https://github.com/duckduckgo/privacy-configuration/issues/326"
+                },
+                {
                     "domain": "motherdenim.com",
                     "reason": "https://github.com/duckduckgo/privacy-configuration/issues/326"
                 },
                 {
latest
v4/android-config.json (13 more)
  • v4/android-config.json
  • v4/extension-brave-config.json
  • v4/extension-bravemv3-config.json
  • v4/extension-chrome-config.json
  • v4/extension-chromemv3-config.json
  • v4/extension-config.json
  • v4/extension-edg-config.json
  • v4/extension-edge-config.json
  • v4/extension-edgmv3-config.json
  • v4/extension-firefox-config.json
  • v4/extension-safarimv3-config.json
  • v4/ios-config.json
  • v4/macos-config.json
  • v4/windows-config.json
--- v4/windows-config.json
+++ v4/windows-config.json
@@ -131,8 +131,11 @@
                 {
                     "domain": "ksta.de"
                 },
                 {
+                    "domain": "larazon.es"
+                },
+                {
                     "domain": "motherdenim.com"
                 },
                 {
                     "domain": "nationalgrid.co.uk"

@muodov muodov added this pull request to the merge queue Apr 23, 2024
Merged via the queue into main with commit 35598e4 Apr 23, 2024
12 checks passed
@muodov muodov deleted the tagawa/larazon-es branch April 23, 2024 07:46
@tagawa
Copy link
Contributor Author

tagawa commented Apr 23, 2024

Thanks @muodov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge when ready If you approve this PR, merge it immediately
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants