fix: remove updated_at from app_settings inserts in multiple migrations
Mirror to GitHub / mirror (push) Successful in 32s
Test and Lint / backend-test (push) Successful in 1m25s
continuous-integration/drone/push Build is passing
Test and Lint / frontend-test (push) Successful in 2m2s
Version and Release / version-bump (push) Successful in 36s
Version and Release / trigger-drone (push) Successful in 3s
Mirror to GitHub / mirror (push) Successful in 32s
Test and Lint / backend-test (push) Successful in 1m25s
continuous-integration/drone/push Build is passing
Test and Lint / frontend-test (push) Successful in 2m2s
Version and Release / version-bump (push) Successful in 36s
Version and Release / trigger-drone (push) Successful in 3s
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 <noreply@anthropic.com>
This commit is contained in:
@@ -8,8 +8,7 @@ exports.up = async function(knex) {
|
|||||||
await knex('app_settings').insert({
|
await knex('app_settings').insert({
|
||||||
setting_key: 'general_default_welcome_message',
|
setting_key: 'general_default_welcome_message',
|
||||||
setting_value: JSON.stringify('Thank you for using our photo sharing service! We hope you enjoy your photos.'),
|
setting_value: JSON.stringify('Thank you for using our photo sharing service! We hope you enjoy your photos.'),
|
||||||
setting_type: 'general',
|
setting_type: 'general'
|
||||||
updated_at: new Date()
|
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -41,8 +41,7 @@ exports.up = async function(knex) {
|
|||||||
|
|
||||||
if (!exists) {
|
if (!exists) {
|
||||||
await knex('app_settings').insert({
|
await knex('app_settings').insert({
|
||||||
...setting,
|
...setting
|
||||||
updated_at: knex.fn.now()
|
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -87,8 +87,7 @@ exports.up = async function(knex) {
|
|||||||
{
|
{
|
||||||
setting_key: 'feedback_notification_email',
|
setting_key: 'feedback_notification_email',
|
||||||
setting_value: JSON.stringify(''),
|
setting_value: JSON.stringify(''),
|
||||||
setting_type: 'feedback',
|
setting_type: 'feedback'
|
||||||
updated_at: new Date()
|
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
setting_key: 'feedback_rate_limits',
|
setting_key: 'feedback_rate_limits',
|
||||||
@@ -97,8 +96,7 @@ exports.up = async function(knex) {
|
|||||||
comment: { max: 20, window: 3600 }, // 20 comments per hour
|
comment: { max: 20, window: 3600 }, // 20 comments per hour
|
||||||
like: { max: 200, window: 3600 } // 200 likes per hour
|
like: { max: 200, window: 3600 } // 200 likes per hour
|
||||||
}),
|
}),
|
||||||
setting_type: 'feedback',
|
setting_type: 'feedback'
|
||||||
updated_at: new Date()
|
|
||||||
}
|
}
|
||||||
]);
|
]);
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user