Update app.py

app.py

@@ -629,21 +629,26 @@ def create_demo():
                     "content": "I apologize, but I'm experiencing technical difficulties. For emergencies, please call 999."
                 }]
 
-        # Add these event handlers
-    feedback_positive.click(
-        lambda history: process_feedback(True, feedback_text.value, history),
-        inputs=[chatbot],
-        outputs=[feedback_text]
-    )
-    
-    feedback_negative.click(
-        lambda history: process_feedback(False, feedback_text.value, history),
-        inputs=[chatbot],
-        outputs=[feedback_text]
-    )
-    
-    # Add clear chat functionality
-    clear.click(lambda: None, None, chatbot)
+        def process_feedback(positive: bool, comment: str, history: list):
+            try:
+                if not history or len(history) < 2:
+                    return gr.update(value="")
+                
+                last_user_msg = history[-2]["content"] if isinstance(history[-2], dict) else history[-2][0]
+                last_bot_msg = history[-1]["content"] if isinstance(history[-1], dict) else history[-1][1]
+                
+                bot.handle_feedback(
+                    message=last_user_msg,
+                    response=last_bot_msg,
+                    feedback=1 if positive else -1
+                )
+                
+                return gr.update(value="")
+            except Exception as e:
+                logger.error(f"Error processing feedback: {e}")
+                return gr.update(value="")
+        
+
         # Create enhanced Gradio interface
         with gr.Blocks(theme=gr.themes.Soft(
             primary_hue="blue",
@@ -706,6 +711,33 @@ def create_demo():
                     }
                 </style>
             """)
+            # Event Handlers - Moved inside the gr.Blocks context
+            msg.submit(chat, [msg, chatbot], [chatbot]).then(
+                lambda: gr.update(value=""), None, [msg]
+            )
+            
+            submit.click(chat, [msg, chatbot], [chatbot]).then(
+                lambda: gr.update(value=""), None, [msg]
+            )
+            
+            # Feedback handlers
+            feedback_positive.click(
+                lambda h: process_feedback(True, feedback_text.value, h),
+                inputs=[chatbot],
+                outputs=[feedback_text]
+            )
+            
+            feedback_negative.click(
+                lambda h: process_feedback(False, feedback_text.value, h),
+                inputs=[chatbot],
+                outputs=[feedback_text]
+            )
+            
+            # Clear chat
+            clear.click(lambda: None, None, chatbot)
+
+            # Add queue for handling multiple users
+            demo.queue(concurrency_count=1, max_size=10)
 
             # Emergency Banner
             gr.HTML("""
@@ -820,17 +852,7 @@ def create_demo():
                     - Prescription renewals
                 """)
 
-            # Event Handlers
-            msg.submit(chat, [msg, chatbot], [chatbot]).then(
-                lambda: gr.update(value=""), None, [msg]
-            )
-            
-            submit.click(chat, [msg, chatbot], [chatbot]).then(
-                lambda: gr.update(value=""), None, [msg]
-            )
-
-            # Add queue for handling multiple users
-            demo.queue(concurrency_count=1, max_size=10)
+        
 
         return demo