Fix regex pattern in PdfTextSplitter to exclude additional characters; update feedback state handling in Answer component

Author: Kingshuk-Microsoft

scripts/data_utils.py

@@ -161,7 +161,7 @@ def find_urls(string):
                 r"(?i)\b("
                 r"(?:https?://|www\d{0,3}[.]|[a-z0-9.\-]+[.][a-z]{2,4}/)"
                 r"(?:[^()\s<>]+|\(([^()\s<>]+|(\([^()\s<>]+\)))*\))+"
-                r"(?:\(([^()\s<>]+|(\([^()\s<>]+\)))*\)|[^()\s`!\[\]{};:'\".,<>?«»“”‘’])"
+                r"(?:\(([^()\s<>]+|(\([^()\s<>]+\)))*\)|[^()\s`!()\[\]{};:'\".,<>?«»“”‘’])"
                 r")"
             )
             urls = re.findall(regex, string)

src/frontend/src/components/Answer/Answer.tsx

@@ -155,7 +155,7 @@ export const Answer = ({ answer, onCitationClicked }: Props) => {
   const onLikeResponseClicked = async () => {
     if (answer.message_id == undefined) return
 
-    let newFeedbackState: Feedback.Neutral | Feedback.Positive | Feedback.Negative
+    let newFeedbackState: Feedback.Neutral | Feedback.Positive
     // Set or unset the thumbs up state
     if (feedbackState == Feedback.Positive) {
       newFeedbackState = Feedback.Neutral