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

Twitter Timeline: add DNT option via filter. #1

Closed
wants to merge 6 commits into from

Conversation

jeherve
Copy link

@jeherve jeherve commented Sep 7, 2018

See Automattic#9359

Here is a different take on Automattic#9359. It seems simpler to me, and does not make any changes to defaults or the update function since we do not have any UI for that setting in the widget settings.

I have also set the option to false by default, to match Twitter's own settings.

@jeherve
Copy link
Author

jeherve commented Dec 6, 2018

closing as I have now merged the changes into Automattic#9359 directly.

@jeherve jeherve closed this Dec 6, 2018
@jeherve jeherve deleted the update/i6722 branch December 6, 2018 14:01
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.

1 participant