fix: remove updated_at from app_settings inserts in multiple migrations
The app_settings table in production doesn't have created_at/updated_at columns. Fixed inconsistent usage across migrations: - Migration 014: removed updated_at: new Date() - Migration 027: removed updated_at: knex.fn.now() - Migration 033: removed updated_at: new Date() This ensures all migrations are consistent and won't fail in production. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
This commit is contained in:
@@ -87,8 +87,7 @@ exports.up = async function(knex) {
|
||||
{
|
||||
setting_key: 'feedback_notification_email',
|
||||
setting_value: JSON.stringify(''),
|
||||
setting_type: 'feedback',
|
||||
updated_at: new Date()
|
||||
setting_type: 'feedback'
|
||||
},
|
||||
{
|
||||
setting_key: 'feedback_rate_limits',
|
||||
@@ -97,8 +96,7 @@ exports.up = async function(knex) {
|
||||
comment: { max: 20, window: 3600 }, // 20 comments per hour
|
||||
like: { max: 200, window: 3600 } // 200 likes per hour
|
||||
}),
|
||||
setting_type: 'feedback',
|
||||
updated_at: new Date()
|
||||
setting_type: 'feedback'
|
||||
}
|
||||
]);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user