From 0a7a313e5dd33e092b5dece90b66788491e9d52d Mon Sep 17 00:00:00 2001
From: ManishMadan2882 <manishmadan321@gmail.com>
Date: Tue, 26 Nov 2024 18:57:11 +0530
Subject: [PATCH 1/5] (fix:conv) input touches viewport bottom in mobile

---
 frontend/src/conversation/Conversation.tsx | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/frontend/src/conversation/Conversation.tsx b/frontend/src/conversation/Conversation.tsx
index ed69064ac..7f06d6b1c 100644
--- a/frontend/src/conversation/Conversation.tsx
+++ b/frontend/src/conversation/Conversation.tsx
@@ -15,7 +15,6 @@ import { useDarkTheme, useMediaQuery } from '../hooks';
 import { ShareConversationModal } from '../modals/ShareConversationModal';
 import { selectConversationId } from '../preferences/preferenceSlice';
 import { AppDispatch } from '../store';
-import conversationService from '../api/services/conversationService';
 import ConversationBubble from './ConversationBubble';
 import { handleSendFeedback } from './conversationHandlers';
 import { FEEDBACK, Query } from './conversationModels';
@@ -323,8 +322,8 @@ export default function Conversation() {
         )}
       </div>
 
-      <div className="flex w-11/12 flex-col items-end self-center rounded-2xl bg-opacity-0 z-3 sm:w-[62%] h-auto">
-        <div className="flex w-full items-center rounded-[40px] border border-silver bg-white py-1 dark:bg-raisin-black">
+      <div className="flex w-11/12 flex-col items-end self-center rounded-2xl bg-opacity-0 z-3 sm:w-[62%] h-auto py-1">
+        <div className="flex w-full items-center rounded-[40px] border border-silver bg-white dark:bg-raisin-black">
           <textarea
             id="inputbox"
             ref={inputRef}

From a86b342ba5a2d829fb484105264b32350a2cb5cd Mon Sep 17 00:00:00 2001
From: ManishMadan2882 <manishmadan321@gmail.com>
Date: Tue, 26 Nov 2024 19:14:50 +0530
Subject: [PATCH 2/5] fix(bubble) smaller fonts on mobile questions

---
 frontend/src/conversation/ConversationBubble.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/conversation/ConversationBubble.tsx b/frontend/src/conversation/ConversationBubble.tsx
index 567b09e91..0e9122507 100644
--- a/frontend/src/conversation/ConversationBubble.tsx
+++ b/frontend/src/conversation/ConversationBubble.tsx
@@ -95,7 +95,7 @@ const ConversationBubble = forwardRef<
               style={{
                 wordBreak: 'break-word',
               }}
-              className="ml-2 mr-2 flex items-center rounded-[28px] bg-purple-30 py-[14px] px-[19px] text-white max-w-full whitespace-pre-wrap leading-normal"
+              className="text-sm sm:text-base ml-2 mr-2 flex items-center rounded-[28px] bg-purple-30 py-[14px] px-[19px] text-white max-w-full whitespace-pre-wrap leading-normal"
             >
               {message}
             </div>

From a8231d375aaa0e75a3322cfb2679ed30ba100a36 Mon Sep 17 00:00:00 2001
From: ManishMadan2882 <manishmadan321@gmail.com>
Date: Wed, 27 Nov 2024 00:45:01 +0530
Subject: [PATCH 3/5] (fix:markdown) code overflows

---
 frontend/src/conversation/ConversationBubble.tsx | 5 +----
 frontend/src/conversation/conversationSlice.ts   | 3 +--
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/frontend/src/conversation/ConversationBubble.tsx b/frontend/src/conversation/ConversationBubble.tsx
index 0e9122507..1eee41e78 100644
--- a/frontend/src/conversation/ConversationBubble.tsx
+++ b/frontend/src/conversation/ConversationBubble.tsx
@@ -337,10 +337,7 @@ const ConversationBubble = forwardRef<
                       </div>
                     </div>
                   ) : (
-                    <code
-                      className={className ? className : 'whitespace-pre-line'}
-                      {...props}
-                    >
+                    <code className="whitespace-pre-line" {...props}>
                       {children}
                     </code>
                   );
diff --git a/frontend/src/conversation/conversationSlice.ts b/frontend/src/conversation/conversationSlice.ts
index 5e1f9b27f..9298e7a52 100644
--- a/frontend/src/conversation/conversationSlice.ts
+++ b/frontend/src/conversation/conversationSlice.ts
@@ -157,8 +157,7 @@ export const fetchAnswer = createAsyncThunk<
     result: '',
     sources: [],
   };
-},
-);
+});
 
 export const conversationSlice = createSlice({
   name: 'conversation',

From bfeae3a95b077b4f045d8acd7694e651fc67f163 Mon Sep 17 00:00:00 2001
From: ManishMadan2882 <manishmadan321@gmail.com>
Date: Wed, 27 Nov 2024 16:54:05 +0530
Subject: [PATCH 4/5] (fix:settings) truncate long doc names

---
 frontend/src/settings/Documents.tsx | 34 ++++++++++++++++++-----------
 frontend/src/utils/stringUtils.ts   |  4 ++++
 2 files changed, 25 insertions(+), 13 deletions(-)
 create mode 100644 frontend/src/utils/stringUtils.ts

diff --git a/frontend/src/settings/Documents.tsx b/frontend/src/settings/Documents.tsx
index 8e563f65a..cea14f0a7 100644
--- a/frontend/src/settings/Documents.tsx
+++ b/frontend/src/settings/Documents.tsx
@@ -15,6 +15,7 @@ import { Doc, DocumentsProps, ActiveState } from '../models/misc'; // Ensure Act
 import { getDocs, getDocsWithPagination } from '../preferences/preferenceApi';
 import { setSourceDocs } from '../preferences/preferenceSlice';
 import { setPaginatedDocuments } from '../preferences/preferenceSlice';
+import { truncate } from '../utils/stringUtils';
 
 // Utility function to format numbers
 const formatTokens = (tokens: number): string => {
@@ -134,7 +135,6 @@ const Documents: React.FC<DocumentsProps> = ({
   };
 
   useEffect(() => {
-    // console.log('modalState', modalState);
     if (modalState === 'INACTIVE') {
       refreshDocs(sortField, currentPage, rowsPerPage);
     }
@@ -184,13 +184,13 @@ const Documents: React.FC<DocumentsProps> = ({
             <div className="flex flex-col">
               <div className="flex-grow">
                 <div className="dark:border-silver/40 border-silver rounded-md border overflow-auto">
-                  <table className="min-w-full divide-y divide-silver dark:divide-silver/40 ">
+                  <table className="min-w-full divide-y divide-silver dark:divide-silver/40 text-xs sm:text-sm ">
                     <thead>
                       <tr className="text-nowrap">
-                        <th className="px-5 py-3 text-start text-sm font-medium text-gray-700 dark:text-gray-50 uppercase">
+                        <th className="px-5 py-3 text-start font-medium text-gray-700 dark:text-gray-50 uppercase w-96">
                           {t('settings.documents.name')}
                         </th>
-                        <th className="px-5 py-3 text-start text-sm font-medium text-gray-700 dark:text-gray-50 uppercase">
+                        <th className="px-5 py-3 text-start font-medium text-gray-700 dark:text-gray-50 uppercase">
                           <div className="flex justify-center items-center">
                             {t('settings.documents.date')}
                             <img
@@ -201,7 +201,10 @@ const Documents: React.FC<DocumentsProps> = ({
                             />
                           </div>
                         </th>
-                        <th className="px-5 py-2 text-center text-sm font-medium text-gray-700 dark:text-gray-50 uppercase">
+                        <th
+                          scope="col"
+                          className="px-5 py-2 text-center font-medium text-gray-700 dark:text-gray-50 uppercase"
+                        >
                           <div className="flex justify-center items-center">
                             {t('settings.documents.tokenUsage')}
                             <img
@@ -221,16 +224,18 @@ const Documents: React.FC<DocumentsProps> = ({
                         */}
                         <th
                           scope="col"
-                          className="px-6 py-2 text-start text-xs font-medium text-gray-700 dark:text-gray-50 uppercase"
-                        ></th>
+                          className="px-6 py-2 text-start font-medium text-gray-700 dark:text-gray-50 uppercase"
+                        >
+                          {' '}
+                        </th>
                       </tr>
                     </thead>
                     <tbody className="divide-y divide-gray-200 dark:divide-neutral-700">
                       {!currentDocuments?.length && (
                         <tr>
                           <td
-                            colSpan={5}
-                            className="!p-4 text-gray-800 dark:text-neutral-200 text-center"
+                            colSpan={4}
+                            className="!py-4 text-gray-800 dark:text-neutral-200 text-center"
                           >
                             {t('settings.documents.noData')}
                           </td>
@@ -239,13 +244,16 @@ const Documents: React.FC<DocumentsProps> = ({
                       {Array.isArray(currentDocuments) &&
                         currentDocuments.map((document, index) => (
                           <tr key={index} className="text-nowrap font-normal">
-                            <td className="px-6 py-4 whitespace-nowrap text-left text-sm font-medium text-gray-800 dark:text-neutral-200">
-                              {document.name}
+                            <td
+                              title={document.name}
+                              className="px-6 py-4 whitespace-nowrap text-left font-medium text-gray-800 dark:text-neutral-200"
+                            >
+                              {truncate(document.name, 50)}
                             </td>
-                            <td className="px-6 py-4 whitespace-nowrap text-center text-sm font-medium text-gray-800 dark:text-neutral-200">
+                            <td className="px-6 py-4 whitespace-nowrap text-center font-medium text-gray-800 dark:text-neutral-200">
                               {document.date}
                             </td>
-                            <td className="px-6 py-4 whitespace-nowrap text-center text-sm font-medium text-gray-800 dark:text-neutral-200">
+                            <td className="px-6 py-4 whitespace-nowrap text-center font-medium text-gray-800 dark:text-neutral-200">
                               {document.tokens
                                 ? formatTokens(+document.tokens)
                                 : ''}
diff --git a/frontend/src/utils/stringUtils.ts b/frontend/src/utils/stringUtils.ts
new file mode 100644
index 000000000..e87a7af38
--- /dev/null
+++ b/frontend/src/utils/stringUtils.ts
@@ -0,0 +1,4 @@
+export function truncate(str: string, n: number) {
+  // slices long strings and ends with ...
+  return str.length > n ? str.slice(0, n - 1) + '...' : str;
+}

From 0a97e5b7beaa420262f3fc0671aaeffe172e9947 Mon Sep 17 00:00:00 2001
From: ManishMadan2882 <manishmadan321@gmail.com>
Date: Wed, 27 Nov 2024 16:57:47 +0530
Subject: [PATCH 5/5] (fix:delete_old) del source from db even when
 FileNotFound err

---
 application/api/user/routes.py | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/application/api/user/routes.py b/application/api/user/routes.py
index b6bd5feac..fa0ff3828 100644
--- a/application/api/user/routes.py
+++ b/application/api/user/routes.py
@@ -248,13 +248,12 @@ def get(self):
                 jsonify({"success": False, "message": "Missing required fields"}), 400
             )
 
-        try:
-            doc = sources_collection.find_one(
+        doc = sources_collection.find_one(
                 {"_id": ObjectId(source_id), "user": "local"}
-            )
-            if not doc:
+        )
+        if not doc:
                 return make_response(jsonify({"status": "not found"}), 404)
-
+        try:
             if settings.VECTOR_STORE == "faiss":
                 shutil.rmtree(os.path.join(current_dir, "indexes", str(doc["_id"])))
             else:
@@ -263,12 +262,12 @@ def get(self):
                 )
                 vectorstore.delete_index()
 
-            sources_collection.delete_one({"_id": ObjectId(source_id)})
         except FileNotFoundError:
             pass
         except Exception as err:
             return make_response(jsonify({"success": False, "error": str(err)}), 400)
-
+        
+        sources_collection.delete_one({"_id": ObjectId(source_id)})
         return make_response(jsonify({"success": True}), 200)