modify the timestamp of veiwed history

Author: elainefan331

backend/migrations/20251028184721-create-view-history.js

@@ -33,14 +33,18 @@ module.exports = {
       viewed_at: {
         type: Sequelize.DATE,
         allowNull: false,
-        defaultValue: Sequelize.literal("CURRENT_TIMESTAMP"),
       },
     });
 
     // Add indexes for common queries
     await queryInterface.addIndex("view_history", ["user_id"]);
     await queryInterface.addIndex("view_history", ["dataset_id"]);
     await queryInterface.addIndex("view_history", ["viewed_at"]);
+    await queryInterface.addConstraint("view_history", {
+      fields: ["user_id", "dataset_id"],
+      type: "unique",
+      name: "unique_user_dataset_view",
+    });
   },
 
   async down(queryInterface, Sequelize) {

backend/src/controllers/activity.controller.js

@@ -305,24 +305,42 @@ const incrementViewCount = async (dbName, datasetId) => {
 
 // function2: add or update a viewed history
 const trackUserHistory = async (userId, datasetId) => {
+  console.log("=== trackUserHistory called ===");
+  console.log("userId:", userId);
+  console.log("datasetId:", datasetId);
   const existingView = await ViewHistory.findOne({
     where: {
       user_id: userId,
       dataset_id: datasetId,
     },
   });
 
+  console.log("existingView found:", !!existingView);
+
   if (existingView) {
+    console.log("OLD viewed_at:", existingView.viewed_at);
     existingView.viewed_at = new Date();
+    console.log("NEW viewed_at (before save):", existingView.viewed_at);
+    // Tell Sequelize this field changed
+    existingView.changed("viewed_at", true);
+    console.log("Changed fields:", existingView.changed());
     await existingView.save();
-    return { isNew: false };
+    console.log("After save, viewed_at:", existingView.viewed_at);
+
+    await existingView.reload();
+    console.log("After reload, viewed_at:", existingView.viewed_at);
+
+    return { isNew: false, viewed_at: existingView.viewed_at };
   } else {
     // create new record
-    await ViewHistory.create({
+    console.log("Creating new view record");
+    const newView = await ViewHistory.create({
       user_id: userId,
       dataset_id: datasetId,
+      viewed_at: new Date(),
     });
-    return { isNew: true };
+    console.log("Created view with viewed_at:", newView.viewed_at);
+    return { isNew: true, viewed_at: newView.viewed_at };
   }
 };
 
@@ -344,6 +362,7 @@ const trackView = async (req, res) => {
       viewCount: dataset.views_count,
       personalHistoryTracked: !!historyResult,
       isNewView: historyResult?.isNew,
+      viewed_at: historyResult?.viewed_at,
       isAuthenticated: !!user,
     });
   } catch (error) {

backend/src/models/index.js

@@ -162,16 +162,17 @@ ViewHistory.init(
     },
     viewed_at: {
       type: DataTypes.DATE,
-      defaultValue: DataTypes.NOW,
+      // defaultValue: DataTypes.NOW,
     },
   },
   {
     sequelize,
     tableName: "view_history",
-    timestamps: true,
+    // timestamps: true,
+    timestamps: false,
     underscored: true,
-    createdAt: "viewed_at",
-    updatedAt: false,
+    // createdAt: "viewed_at",
+    // updatedAt: false,
   }
 );