From f6c39b3d9baf15e07b5becea7b4a06fc88c676b5 Mon Sep 17 00:00:00 2001
From: Uday Sagar <udaysagar.mail@gmail.com>
Date: Fri, 12 Jul 2024 21:20:17 +0530
Subject: [PATCH] code review

---
 .../Facility/DoctorNoteReplyPreviewCard.tsx   |  7 ++--
 src/Components/Facility/PatientNoteCard.tsx   | 35 +++++++++----------
 2 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/src/Components/Facility/DoctorNoteReplyPreviewCard.tsx b/src/Components/Facility/DoctorNoteReplyPreviewCard.tsx
index aa673fafee6..c99f0bf2118 100644
--- a/src/Components/Facility/DoctorNoteReplyPreviewCard.tsx
+++ b/src/Components/Facility/DoctorNoteReplyPreviewCard.tsx
@@ -16,7 +16,10 @@ const DoctorNoteReplyPreviewCard = ({
   children,
   cancelReply,
 }: Props) => {
-  return parentNote ? (
+  if (!parentNote) {
+    return children;
+  }
+  return (
     <div className="mt-3 flex flex-col rounded-lg border border-gray-300 bg-gray-200 pt-2 text-gray-800">
       <div className="flex flex-col">
         <div className="flex justify-between pl-3">
@@ -64,8 +67,6 @@ const DoctorNoteReplyPreviewCard = ({
       </div>
       <div>{children}</div>
     </div>
-  ) : (
-    <div>{children}</div>
   );
 };
 
diff --git a/src/Components/Facility/PatientNoteCard.tsx b/src/Components/Facility/PatientNoteCard.tsx
index aa9bfb6b3ed..f65a45dc764 100644
--- a/src/Components/Facility/PatientNoteCard.tsx
+++ b/src/Components/Facility/PatientNoteCard.tsx
@@ -290,27 +290,26 @@ const PatientNoteCard = ({
               >
                 <MarkdownPreview markdown={noteField} />
                 <div className="flex gap-2">
-                  {note.files &&
-                    note.files.length > 0 &&
-                    note.files.map((file: any) => (
+                  {/* temporaryly marked as any due to external_id */}
+                  {note?.files?.map((file: any) => (
+                    <div
+                      key={file.id}
+                      className="relative mt-1 h-20 w-20 cursor-pointer rounded-md bg-gray-100 shadow-sm hover:bg-gray-200"
+                    >
                       <div
-                        key={file.id}
-                        className="relative mt-1 h-20 w-20 cursor-pointer rounded-md bg-gray-100 shadow-sm hover:bg-gray-200"
+                        className="flex h-full w-full flex-col items-center justify-center p-2"
+                        onClick={() => loadFile(file.external_id, note.id)}
                       >
-                        <div
-                          className="flex h-full w-full flex-col items-center justify-center p-2"
-                          onClick={() => loadFile(file.external_id, note.id)}
-                        >
-                          <CareIcon
-                            icon="l-file"
-                            className="shrink-0 text-2xl text-gray-600"
-                          />
-                          <span className="mt-1 max-h-[2.5em] w-full overflow-hidden text-ellipsis break-words text-center text-xs text-gray-600">
-                            {file.name}
-                          </span>
-                        </div>
+                        <CareIcon
+                          icon="l-file"
+                          className="shrink-0 text-2xl text-gray-600"
+                        />
+                        <span className="mt-1 max-h-[2.5em] w-full overflow-hidden text-ellipsis break-words text-center text-xs text-gray-600">
+                          {file.name}
+                        </span>
                       </div>
-                    ))}
+                    </div>
+                  ))}
                 </div>
                 {mode == "thread-view" && note.replies.length > 0 && (
                   <div className="mt-2 flex items-center text-xs text-gray-500">