Explain why we set updatedAt manually

owid · Nov 1, 2024 · 8d720bb · 8d720bb
1 parent 26540f1
commit 8d720bb
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/db/model/Variable.ts b/db/model/Variable.ts
@@ -397,6 +397,9 @@ export async function updateGrapherConfigETLOfVariable(
         variableId,
     })
 
+    // Set the updatedAt manually instead of letting the DB do it so it is the
+    // same across different tables. The inconsistency caused issues in the
+    // past in chart-sync.
     const now = new Date()
 
     if (variable.etl) {
@@ -478,6 +481,9 @@ export async function updateGrapherConfigAdminOfVariable(
         patchConfigAdmin
     )
 
+    // Set the updatedAt manually instead of letting the DB do it so it is the
+    // same across different tables. The inconsistency caused issues in the
+    // past in chart-sync.
     const now = new Date()
 
     if (variable.admin) {