From a8d7d90ef4419abb1c6f3f000deb7b2f3a3951fb Mon Sep 17 00:00:00 2001 From: Aaro Koivukangas Date: Tue, 15 Sep 2026 13:12:00 +0300 Subject: [PATCH] [Updater] Change error to warning with missing teacher credit_id --- updater/sis-updater-worker/src/db/index.js | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/updater/sis-updater-worker/src/db/index.js b/updater/sis-updater-worker/src/db/index.js index a8b119653..bcf628f74 100644 --- a/updater/sis-updater-worker/src/db/index.js +++ b/updater/sis-updater-worker/src/db/index.js @@ -162,7 +162,16 @@ export const bulkCreate = async ( error.name === 'SequelizeForeignKeyConstraintError' && error.parent.constraint === 'credit_course_id_fkey' ) { - logger.warn('Skipping credit with course_id pointing to non-existent course', { entity }) + logger.warn('Skipping credit with course_id pointing to non-existent course', { + attainment_id: entity?.id, + course_id: entity?.course_id, + }) + // Skip errors where that credit was not created and other rows in teacher_credits reference it + } else if ( + error.name === 'SequelizeForeignKeyConstraintError' && + error.parent.constraint === 'credit_teachers_credit_id_fkey' + ) { + logger.warn("Skipping adding teacher's credit with missing credit_id", { entity }) } else { logger.error(`Single-entity upsert failed. ${error.name?.startsWith('Sequelize') ? error.toString() : ''}`, { error,