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 viewNsfwGuilds to User Settings #1043

Closed
wants to merge 15 commits into from

Conversation

bitfl0wer
Copy link

https://luna.gitlab.io/discord-unofficial-docs/user_settings.html says that there should be a user setting called view_nsfw_guilds. This did not exist yet, so I've created it.

I don't know if I did the migrations thingy correctly, though. They do work for me, however, I am not sure if this is everything I needed to do?

Copy link
Contributor

@erkinalp erkinalp left a comment

Choose a reason for hiding this comment

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

in mysql and psql, alter table can add and drop columns

@MaddyUnderStars MaddyUnderStars linked an issue Apr 24, 2023 that may be closed by this pull request
@MaddyUnderStars
Copy link
Member

TODO: fix migrations and why was database.db pushed

@bitfl0wer
Copy link
Author

uhhhhhhh oops

@Puyodead1
Copy link
Contributor

why is that not in gitignore???

@MaddyUnderStars
Copy link
Member

it is

@Puyodead1
Copy link
Contributor

it is

then how did it get commited?

@MaddyUnderStars
Copy link
Member

I don't know but it's in the gitignore. I just checked

@Puyodead1
Copy link
Contributor

oh ma gawd what da hellllll

@bitfl0wer
Copy link
Author

What if I tell you I did it on purpose

@Puyodead1
Copy link
Contributor

What if I tell you I did it on purpose

I would have words with you

@bitfl0wer
Copy link
Author

I actually did it on purpose, because I forgot that this pr was open on that fork, and I now use my fork for CD. The database contains a spacebar instance with some pre-defined guilds, users and settings for me to test with. I use this, so I can just clone the whole thing and use it in github actions or whatever

@bitfl0wer bitfl0wer closed this Aug 10, 2023
@bitfl0wer bitfl0wer deleted the master branch August 10, 2023 22:28
@bitfl0wer bitfl0wer restored the master branch August 10, 2023 22:29
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.

UserSettings are outdated
4 participants