Merge pull request ansari-project#167 from ansari-project/feedback-bug-fix

waleedkadous · web-flow · commit 96d11647e7ac · 2025-04-08T16:08:13.000-07:00
Resolve thread feedback setting bug
diff --git a/src/ansari/ansari_db.py b/src/ansari/ansari_db.py
@@ -337,8 +337,12 @@ def add_feedback(self, user_id, thread_id, message_id, feedback_class, comment):
                 "updated_at": datetime.now(timezone.utc),
             }
 
-            self.get_collection("threads").update_one(
-                {"_id": ObjectId(thread_id), "messages.id": ObjectId(message_id)}, {"$set": {"messages.$.feedback": feedback}}
+            self.get_collection("threads").update_one({
+                    "_id": ObjectId(thread_id),
+                    "user_id": ObjectId(user_id),
+                    "messages.id": message_id
+                },
+                {"$set": {"messages.$.feedback": feedback}}
             )
 
             return {"status": "success"}

Original file line number	Diff line number	Diff line change
`@@ -337,8 +337,12 @@ def add_feedback(self, user_id, thread_id, message_id, feedback_class, comment):`
`337`	`337`	`"updated_at": datetime.now(timezone.utc),`
`338`	`338`	`}`
`339`	`339`
`340`		`- self.get_collection("threads").update_one(`
`341`		`- {"_id": ObjectId(thread_id), "messages.id": ObjectId(message_id)}, {"$set": {"messages.$.feedback": feedback}}`
	`340`	`+ self.get_collection("threads").update_one({`
	`341`	`+ "_id": ObjectId(thread_id),`
	`342`	`+ "user_id": ObjectId(user_id),`
	`343`	`+ "messages.id": message_id`
	`344`	`+ },`
	`345`	`+ {"$set": {"messages.$.feedback": feedback}}`
`342`	`346`	`)`
`343`	`347`
`344`	`348`	`return {"status": "success"}`