groot/automatisch
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #1969 from automatisch/fix-migrations-rollback

Browse Source 
fix: Remove step constraint down migration
This commit is contained in:
Ali BARIN
2024-07-16 16:04:07 +02:00
committed by GitHub
parent 9304ffffc9 9578cd27dd
commit eac2f729a5
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
packages/backend/src/db/migrations/20220105151725_remove_constraints_from_steps.js
View File

@@ -5,9 +5,11 @@ export async function up(knex) {
});
}
export async function down(knex) {
return knex.schema.alterTable('steps', (table) => {
table.string('key').notNullable().alter();
table.string('app_key').notNullable().alter();
});
export async function down() {
// We can't use down migration here since there are null values which needs to be set!
// We don't want to set those values by default key and app key since it will mislead users.
// return knex.schema.alterTable('steps', (table) => {
// table.string('key').notNullable().alter();
// table.string('app_key').notNullable().alter();
// });
}