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

Add social media blocklists to daemon+CLI #5163

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

faern
Copy link
Member

@faern faern commented Sep 19, 2023

When infra has deployed these block lists to the relays, this setting will allow blocking social media domains directly with the app.

I intentionally left it out of the changelog for now. This should not go into production before it's added to the UI anyway.


This change is Reviewable

@faern faern requested a review from dlon September 19, 2023 11:55
@linear
Copy link

linear bot commented Sep 19, 2023

@faern faern force-pushed the add-daemoncli-support-for-social-media-blocklist-des-354 branch from d92b40a to cee733e Compare September 19, 2023 11:56
Copy link
Member

@dlon dlon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@jbjorkang
Copy link
Member

Will be available on all VPN servers tomorrow afternoon.

@faern faern force-pushed the add-daemoncli-support-for-social-media-blocklist-des-354 branch 2 times, most recently from 36ab1de to 73c0f58 Compare September 20, 2023 08:01
Copy link
Collaborator

@albin-mullvad albin-mullvad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 6 files at r1, 1 of 1 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

When infra has deployed these block lists to the relays,
this setting will allow blocking social media domains
directly with the app
@faern faern force-pushed the add-daemoncli-support-for-social-media-blocklist-des-354 branch from 73c0f58 to 412b799 Compare September 20, 2023 09:09
@faern faern merged commit 070c406 into main Sep 20, 2023
28 of 30 checks passed
@faern faern deleted the add-daemoncli-support-for-social-media-blocklist-des-354 branch September 20, 2023 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants