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

Make nsfw/nsfl opt-in for default/new users #327

Closed
wants to merge 4 commits into from
Closed

Make nsfw/nsfl opt-in for default/new users #327

wants to merge 4 commits into from

Conversation

KyrunaKJ
Copy link
Contributor

Added pref-hidensfw and pref-hidensfl to the settings for base/no users so nsfw and nsfl are opt-in by default.

Added pref-hidensfw and pref-hidensfl to the settings for base/no users so nsfw and nsfl are opt-in by default.
@KyrunaKJ
Copy link
Contributor Author

KyrunaKJ commented Sep 11, 2023

And then tested this with my cookies and my settings persisted so this shouldn't affect any current users.

Initial commit had the fix in the wrong area. Should be good now.

Signed-off-by: KyrunaKJ <[email protected]>
hidensfw and hidensfl set to true

Signed-off-by: KyrunaKJ <[email protected]>
Signed-off-by: KyrunaKJ <[email protected]>
@KyrunaKJ
Copy link
Contributor Author

I was misguided in my earlier change, I've just updated the const.js so that hidensfw and hidensfl are true by default.

@ILiedAboutCake
Copy link
Collaborator

Don’t make off the cuff moderation decisions irreversible code changes for people who sign up during a time period

@KyrunaKJ KyrunaKJ deleted the nsfwdefault branch September 11, 2023 15:56
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.

2 participants