-
-
Notifications
You must be signed in to change notification settings - Fork 885
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
Update DB local_user.theme type to text #3266
Update DB local_user.theme type to text #3266
Conversation
crates/db_schema/src/schema.rs
Outdated
@@ -65,7 +65,7 @@ diesel::table! { | |||
} | |||
|
|||
diesel::table! { | |||
use diesel::sql_types::{Bool, Int4, Nullable, Text, Timestamp, Varchar}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unsure about this and similar changes in this file. They were auto generated... Should I revert them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Diesel CLI is doing this for me too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which command are you running that this happens?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Nutomic diesel migration run
. This change is in my community muting PR as well. It comes from running the migrations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep and also diesel migration redo
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay fixed in #3293
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have reverted those lines in this PR 👍
Updates local_user.theme to type text to match local_site.default_theme
Fixes #3166