-
-
Notifications
You must be signed in to change notification settings - Fork 199
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
Fix Note on Brave Tor Window Setting #1841
Conversation
datoshkr
commented
Oct 15, 2022
•
edited
Loading
edited
- Please check this box to confirm you have disclosed any relevant conflicts of interest in your post.
- Please check this box to confirm your agreement to publish your work under the Creative Commons Attribution-NoDerivatives 4.0 International license, and to grant Privacy Guides a perpetual, worldwide, non-exclusive, transferable, royalty-free, irrevocable license with the right to sublicense such rights through multiple tiers of sublicensees, to reproduce, modify, display, perform and distribute your contribution as part of our project.
🎊 PR Preview 22f4046 has been successfully built and deployed to https://privacyguides-privacyguides-org-preview-pr-1841.surge.sh 🕐 Build time: 78.932s 🤖 By surge-preview |
It took me a bit to understand what this PR was trying to fix, but I eventually got it. A recent PR moved the Tor window section from one section of the settings to another, and we didn't have the necessary code for the note to be handled appropriately. It should now be fine and the same as the "block scripts" option above it. Thank you for spotting this and trying to help fix it! |
This pull request has been mentioned on Privacy Guides. There might be relevant details there: |