From cef2795ed3d1f1771e052d0343820e5036bd3315 Mon Sep 17 00:00:00 2001
From: Shivank Kacker <kacker.shivank@gmail.com>
Date: Mon, 3 Jun 2024 03:58:34 +0530
Subject: [PATCH 01/11] fix button widths

---
 src/CAREUI/interactive/FiltersSlideover.tsx | 2 +-
 src/Components/Common/SortDropdown.tsx      | 3 ++-
 src/Components/Patient/ManagePatients.tsx   | 2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/CAREUI/interactive/FiltersSlideover.tsx b/src/CAREUI/interactive/FiltersSlideover.tsx
index 83f92e2bd90..496f1b3e516 100644
--- a/src/CAREUI/interactive/FiltersSlideover.tsx
+++ b/src/CAREUI/interactive/FiltersSlideover.tsx
@@ -58,7 +58,7 @@ export const AdvancedFilterButton = ({ onClick }: { onClick: () => void }) => {
     <ButtonV2
       ghost
       border
-      className="w-full bg-white sm:w-auto"
+      className="w-full bg-white md:w-auto"
       onClick={onClick}
       id="advanced-filter"
     >
diff --git a/src/Components/Common/SortDropdown.tsx b/src/Components/Common/SortDropdown.tsx
index b29662de0a7..8ffd09ba269 100644
--- a/src/Components/Common/SortDropdown.tsx
+++ b/src/Components/Common/SortDropdown.tsx
@@ -23,8 +23,9 @@ export default function SortDropdownMenu(props: Props) {
     <DropdownMenu
       title={props.label ?? t("sort_by")}
       variant="secondary"
-      className="border border-primary-500 bg-white"
+      className="w-full border border-primary-500 bg-white md:w-auto"
       icon={<CareIcon icon="l-sort" />}
+      containerClassName="w-full md:w-auto"
     >
       {props.options.map(({ isAscending, value }) => (
         <DropdownItem
diff --git a/src/Components/Patient/ManagePatients.tsx b/src/Components/Patient/ManagePatients.tsx
index a99d12c09fc..ae227bbbb6b 100644
--- a/src/Components/Patient/ManagePatients.tsx
+++ b/src/Components/Patient/ManagePatients.tsx
@@ -821,7 +821,7 @@ export const PatientManager = () => {
               selected={qParams.ordering}
               onSelect={updateQuery}
             />
-            <div className="tooltip">
+            <div className="tooltip w-full md:w-auto">
               {!isExportAllowed ? (
                 <ButtonV2
                   onClick={() => {

From 339e2a63c68f7fd4546457c49f925ee9fcc47f73 Mon Sep 17 00:00:00 2001
From: Ashesh <3626859+Ashesh3@users.noreply.github.com>
Date: Tue, 4 Jun 2024 17:33:55 +0000
Subject: [PATCH 02/11] Support retries in Scribe

---
 src/Components/Scribe/Scribe.tsx | 115 ++++++++++++++++++++++---------
 1 file changed, 84 insertions(+), 31 deletions(-)

diff --git a/src/Components/Scribe/Scribe.tsx b/src/Components/Scribe/Scribe.tsx
index 6e876a4762c..b5149267e24 100644
--- a/src/Components/Scribe/Scribe.tsx
+++ b/src/Components/Scribe/Scribe.tsx
@@ -1,4 +1,4 @@
-import React, { useState, useEffect } from "react";
+import React, { useState, useEffect, useRef } from "react";
 import { Popover } from "@headlessui/react";
 import ButtonV2 from "../Common/components/ButtonV2";
 import CareIcon from "../../CAREUI/icons/CareIcon";
@@ -59,7 +59,7 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
   const [open, setOpen] = useState(false);
   const [_progress, setProgress] = useState(0);
   const [stage, setStage] = useState("start");
-  const [editableTranscript, setEditableTranscript] = useState("");
+  const [_editableTranscript, setEditableTranscript] = useState("");
   const [errors, setErrors] = useState<string[]>([]);
   const [isTranscribing, setIsTranscribing] = useState(false);
   const [formFields, setFormFields] = useState<{
@@ -70,6 +70,13 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
   const [isAudioUploading, setIsAudioUploading] = useState(false);
   const [updatedTranscript, setUpdatedTranscript] = useState<string>("");
   const [scribeID, setScribeID] = useState<string>("");
+  const stageRef = useRef(stage);
+
+  useEffect(() => {
+    if (stageRef.current === "cancelled") {
+      setStage("start");
+    }
+  }, [stage]);
 
   const {
     isRecording,
@@ -81,6 +88,7 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
 
   const uploadAudioData = (response: any, audioBlob: Blob) => {
     return new Promise<void>((resolve, reject) => {
+      if (stageRef.current === "cancelled") resolve();
       const url = response.data.signed_url;
       const internal_name = response.data.internal_name;
       const f = audioBlob;
@@ -109,6 +117,7 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
 
   const uploadAudio = async (audioBlob: Blob, associatingId: string) => {
     return new Promise((resolve, reject) => {
+      if (stageRef.current === "cancelled") resolve({});
       const category = "AUDIO";
       const name = "audio.mp3";
       const filename = Date.now().toString();
@@ -127,6 +136,8 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
           uploadAudioData(response, audioBlob)
             .then(() => {
               if (!response?.data?.id) throw Error("Error uploading audio");
+
+              if (stageRef.current === "cancelled") resolve({});
               markUploadComplete(response?.data.id, associatingId).then(() => {
                 resolve(response.data);
               });
@@ -145,12 +156,14 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
     startRecording();
     setProgress(25);
     setStage("recording");
+    stageRef.current = "recording";
   };
 
   const handleStopRecordingClick = () => {
     stopRecording();
     setProgress(50);
     setStage("recording-review");
+    stageRef.current = "recording-end";
   };
 
   const handleEditChange = (e: {
@@ -161,6 +174,7 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
 
   const waitForTranscript = async (externalId: string): Promise<string> => {
     return new Promise((resolve, reject) => {
+      if (stageRef.current === "cancelled") resolve("");
       const interval = setInterval(async () => {
         try {
           const res = await request(routes.getScribe, {
@@ -191,9 +205,29 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
     });
   };
 
+  const reProcessTranscript = async () => {
+    setErrors([]);
+    setEditableTranscript(updatedTranscript);
+    setStage("review");
+    const res = await request(routes.updateScribe, {
+      body: {
+        status: "READY",
+        transcript: updatedTranscript,
+        ai_response: null,
+      },
+      pathParams: {
+        external_id: scribeID,
+      },
+    });
+    if (res.error || !res.data) throw Error("Error updating scribe instance");
+    setStage("review");
+    await handleSubmitTranscript(res.data.external_id);
+  };
+
   const waitForFormData = async (externalId: string): Promise<string> => {
     return new Promise((resolve, reject) => {
       const interval = setInterval(async () => {
+        if (stageRef.current === "cancelled") resolve("");
         try {
           const res = await request(routes.getScribe, {
             pathParams: {
@@ -270,12 +304,14 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
   };
 
   const handleSubmitTranscript = async (external_id: string) => {
+    if (stageRef.current === "cancelled") return;
     setProgress(75);
     setIsGPTProcessing(true);
     try {
       const updatedFieldsResponse = await waitForFormData(external_id);
       setProgress(100);
       const parsedFormData = JSON.parse(updatedFieldsResponse ?? "{}");
+      if (stageRef.current === "cancelled") return;
       setFormFields(parsedFormData);
       onFormUpdate(parsedFormData);
       setStage("final-review");
@@ -286,6 +322,7 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
   };
 
   const startTranscription = async (scribeID: string) => {
+    if (stageRef.current === "cancelled") return;
     setIsTranscribing(true);
     const errors = [];
     try {
@@ -302,6 +339,7 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
 
       //poll for transcript
       const transcriptResponse = await waitForTranscript(res.data.external_id);
+      if (stageRef.current === "cancelled") return;
       setEditableTranscript(transcriptResponse);
       setUpdatedTranscript(transcriptResponse);
       setStage("review");
@@ -329,6 +367,10 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
     setFormFields({});
     setEditableTranscript("");
     setUpdatedTranscript("");
+    setIsAudioUploading(false);
+    setScribeID("");
+    setIsHoveringCancelRecord(false);
+    stageRef.current = "cancelled";
   };
 
   const processFormField = (
@@ -485,44 +527,47 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
                         src={URL.createObjectURL(blob)}
                       />
                     ))}
+                  <ButtonV2
+                    onClick={() => {
+                      handleRerecordClick();
+                    }}
+                    className="w-full"
+                  >
+                    Restart Recording
+                  </ButtonV2>
                 </div>
               )}
 
               {(stage === "review" || stage === "final-review") && (
                 <div className="my-2 w-full">
-                  <p className="mb-2 text-sm font-semibold text-gray-600">
-                    Transcript
-                  </p>
+                  <div className="flex justify-between">
+                    <p className="mb-2 text-sm font-semibold text-gray-600">
+                      Transcript
+                    </p>
+                    <p className="mb-2 text-sm italic text-gray-500">
+                      (Edit if needed)
+                    </p>
+                  </div>
                   <textarea
                     className="h-32 w-full rounded-md border border-gray-300 p-2 font-mono text-sm"
                     value={updatedTranscript}
                     onChange={handleEditChange}
                   />
-                  {updatedTranscript !== editableTranscript && (
-                    <ButtonV2
-                      onClick={async () => {
-                        setErrors([]);
-                        setEditableTranscript(updatedTranscript);
-                        setStage("review");
-                        const res = await request(routes.updateScribe, {
-                          body: {
-                            status: "READY",
-                            transcript: updatedTranscript,
-                          },
-                          pathParams: {
-                            external_id: scribeID,
-                          },
-                        });
-                        if (res.error || !res.data)
-                          throw Error("Error updating scribe instance");
-                        setStage("review");
-                        await handleSubmitTranscript(res.data.external_id);
-                      }}
-                      className="my-2 w-full"
-                    >
-                      Process Transcript
-                    </ButtonV2>
-                  )}
+
+                  <ButtonV2
+                    onClick={reProcessTranscript}
+                    className="my-2 w-full"
+                  >
+                    Process Transcript
+                  </ButtonV2>
+                  <ButtonV2
+                    onClick={async () => {
+                      setStage("recording-review");
+                    }}
+                    className="mb-2 w-full"
+                  >
+                    Regenerate Transcript
+                  </ButtonV2>
                   {stage === "review" && (
                     <p className="animate-pulse text-sm text-gray-500">
                       {getStageMessage(stage)}
@@ -568,10 +613,18 @@ export const Scribe: React.FC<ScribeProps> = ({ fields, onFormUpdate }) => {
                       </div>
                     )}
                   {stage === "final-review" && (
-                    <p className="my-2 text-sm text-gray-600">
+                    <p className="mt-2 text-sm text-gray-600">
                       {getStageMessage(stage)}
                     </p>
                   )}
+                  {stage === "final-review" && !isGPTProcessing && (
+                    <ButtonV2
+                      onClick={reProcessTranscript}
+                      className="mt-2 w-full"
+                    >
+                      Reobtain Form Data
+                    </ButtonV2>
+                  )}
                 </div>
               )}
               <div className="">

From dda7cb98a1fd315d4b3b3aa2f4c8033eed82b5da Mon Sep 17 00:00:00 2001
From: Ashraf Mohammed <98876115+AshrafMd-1@users.noreply.github.com>
Date: Wed, 5 Jun 2024 18:24:16 +0530
Subject: [PATCH 03/11] fix wrong links (#7956)

---
 src/Components/Facility/ConsultationDetails/index.tsx | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/Components/Facility/ConsultationDetails/index.tsx b/src/Components/Facility/ConsultationDetails/index.tsx
index ee1f7bda7ff..e46db70189a 100644
--- a/src/Components/Facility/ConsultationDetails/index.tsx
+++ b/src/Components/Facility/ConsultationDetails/index.tsx
@@ -37,6 +37,7 @@ import PatientInfoCard from "../../Patient/PatientInfoCard";
 import RelativeDateUserMention from "../../Common/RelativeDateUserMention";
 import DiagnosesListAccordion from "../../Diagnosis/DiagnosesListAccordion";
 import { CameraFeedPermittedUserTypes } from "../../../Utils/permissions";
+import Error404 from "../../ErrorPages/404";
 
 const Loading = lazy(() => import("../../Common/Loading"));
 const PageTitle = lazy(() => import("../../Common/PageTitle"));
@@ -68,7 +69,10 @@ const TABS = {
 
 export const ConsultationDetails = (props: any) => {
   const { facilityId, patientId, consultationId } = props;
-  const tab = props.tab.toUpperCase() as keyof typeof TABS;
+  let tab = undefined;
+  if (Object.keys(TABS).includes(props.tab.toUpperCase())) {
+    tab = props.tab.toUpperCase() as keyof typeof TABS;
+  }
   const dispatch: any = useDispatch();
   const [isLoading, setIsLoading] = useState(false);
   const [showDoctors, setShowDoctors] = useState(false);
@@ -194,6 +198,10 @@ export const ConsultationDetails = (props: any) => {
     patientData,
   };
 
+  if (!tab) {
+    return <Error404 />;
+  }
+
   const SelectedTab = TABS[tab];
 
   if (isLoading) {

From 5dfced0583d88adb3a254d6a4aa5a41978e3bcee Mon Sep 17 00:00:00 2001
From: Ashraf Mohammed <98876115+AshrafMd-1@users.noreply.github.com>
Date: Wed, 5 Jun 2024 18:26:36 +0530
Subject: [PATCH 04/11] Fix QR code error notification (#7726)

* fix qr notificaation

* change error notification

* add error notification

* add validation
---
 src/Components/Assets/AssetsList.tsx | 74 ++++++++++++++++++----------
 src/Redux/api.tsx                    |  5 ++
 2 files changed, 53 insertions(+), 26 deletions(-)

diff --git a/src/Components/Assets/AssetsList.tsx b/src/Components/Assets/AssetsList.tsx
index 796e532a41e..f6f83e7030f 100644
--- a/src/Components/Assets/AssetsList.tsx
+++ b/src/Components/Assets/AssetsList.tsx
@@ -105,40 +105,63 @@ const AssetsList = () => {
     prefetch: !!(qParams.facility && qParams.location),
   });
 
-  const getAssetIdFromQR = async (assetUrl: string) => {
+  function isValidURL(url: string) {
+    try {
+      new URL(url);
+      return true;
+    } catch (_) {
+      return false;
+    }
+  }
+
+  const accessAssetIdFromQR = async (assetURL: string) => {
     try {
       setIsLoading(true);
       setIsScannerActive(false);
-      const params = parseQueryParams(assetUrl);
+      if (!isValidURL(assetURL)) {
+        setIsLoading(false);
+        Notification.Error({
+          msg: "Invalid QR code scanned !!!",
+        });
+        return;
+      }
+      const params = parseQueryParams(assetURL);
       // QR Maybe searchParams "asset" or "assetQR"
+      // If no params found, then use assetText
       const assetId = params.asset || params.assetQR;
+
       if (assetId) {
-        const { data } = await request(routes.listAssets, {
-          query: { qr_code_id: assetId },
+        const { data } = await request(routes.listAssetQR, {
+          pathParams: { qr_code_id: assetId },
+        });
+        if (!data) {
+          setIsLoading(false);
+          Notification.Error({
+            msg: "Invalid QR code scanned !!!",
+          });
+          return;
+        }
+        const { data: assetData } = await request(routes.listAssets, {
+          query: { qr_code_id: assetId, limit: 1 },
+        });
+        if (assetData?.results.length === 1) {
+          navigate(
+            `/facility/${assetData.results[0].location_object.facility?.id}/assets/${assetData.results[0].id}`,
+          );
+        } else {
+          setIsLoading(false);
+          Notification.Error({
+            msg: "Asset not found !!!",
+          });
+        }
+      } else {
+        setIsLoading(false);
+        Notification.Error({
+          msg: "Invalid QR code scanned !!!",
         });
-        return data?.results[0].id;
-      }
-    } catch (err) {
-      console.log(err);
-    }
-  };
-
-  const checkValidAssetId = async (assetId: string) => {
-    const { data: assetData } = await request(routes.getAsset, {
-      pathParams: { external_id: assetId },
-    });
-    try {
-      if (assetData) {
-        navigate(
-          `/facility/${assetData.location_object.facility?.id}/assets/${assetId}`,
-        );
       }
     } catch (err) {
       console.log(err);
-      setIsLoading(false);
-      Notification.Error({
-        msg: "Invalid QR code scanned !!!",
-      });
     }
   };
 
@@ -159,8 +182,7 @@ const AssetsList = () => {
         <Scanner
           onResult={async (text) => {
             if (text) {
-              const assetId = await getAssetIdFromQR(text);
-              checkValidAssetId(assetId ?? text);
+              await accessAssetIdFromQR(text);
             }
           }}
           onError={(e) => {
diff --git a/src/Redux/api.tsx b/src/Redux/api.tsx
index f4a9732a40c..439b122ca09 100644
--- a/src/Redux/api.tsx
+++ b/src/Redux/api.tsx
@@ -1286,6 +1286,11 @@ const routes = {
     method: "GET",
     TRes: Type<PaginatedResponse<AvailabilityRecord>>(),
   },
+  listAssetQR: {
+    path: "/api/v1/public/asset_qr/{qr_code_id}/",
+    method: "GET",
+    TRes: Type<AssetData>(),
+  },
 
   // Asset transaction endpoints
 

From 9037e1b131937f30b4aa8d8b6203aebcb39d4a02 Mon Sep 17 00:00:00 2001
From: Mohammed Nihal <57055998+nihal467@users.noreply.github.com>
Date: Thu, 6 Jun 2024 07:53:42 +0530
Subject: [PATCH 05/11] Fix Cypress Flaky test in Investigation (#7979)

* fix flaky cypress test

* fix flaky cypress test
---
 cypress/pageobject/Patient/PatientInvestigation.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cypress/pageobject/Patient/PatientInvestigation.ts b/cypress/pageobject/Patient/PatientInvestigation.ts
index c226c358676..8f73cf908bc 100644
--- a/cypress/pageobject/Patient/PatientInvestigation.ts
+++ b/cypress/pageobject/Patient/PatientInvestigation.ts
@@ -10,7 +10,7 @@ class PatientInvestigation {
   }
 
   selectInvestigation(investigation: string) {
-    cy.get("#search-patient-investigation").click();
+    cy.get("#search-patient-investigation").type(investigation);
     cy.verifyAndClickElement("#investigation-group", investigation);
     cy.verifyAndClickElement("#investigation", "Investigation No. 1");
   }

From 17abf27528748c8a724c96a2bc0d315128e5050f Mon Sep 17 00:00:00 2001
From: Rithvik Nishad <mail@rithviknishad.dev>
Date: Thu, 6 Jun 2024 07:53:54 +0530
Subject: [PATCH 06/11] Show ICD-11 Diagnosis functionality unavailable error
 if issues with Redis (#7973)

---
 .../AddICD11Diagnosis.tsx                           | 13 +++++++++++--
 .../ConsultationDiagnosisEntry.tsx                  |  4 +++-
 src/Components/Diagnosis/DiagnosesListAccordion.tsx | 11 +++++++++--
 src/Components/Facility/ConsultationForm.tsx        |  6 +++++-
 src/Components/Patient/DiagnosesFilter.tsx          | 11 ++++++++++-
 5 files changed, 38 insertions(+), 7 deletions(-)

diff --git a/src/Components/Diagnosis/ConsultationDiagnosisBuilder/AddICD11Diagnosis.tsx b/src/Components/Diagnosis/ConsultationDiagnosisBuilder/AddICD11Diagnosis.tsx
index 0006b454718..2476fa57ab0 100644
--- a/src/Components/Diagnosis/ConsultationDiagnosisBuilder/AddICD11Diagnosis.tsx
+++ b/src/Components/Diagnosis/ConsultationDiagnosisBuilder/AddICD11Diagnosis.tsx
@@ -1,4 +1,4 @@
-import { useState } from "react";
+import { useEffect, useState } from "react";
 import { useTranslation } from "react-i18next";
 import AutocompleteFormField from "../../Form/FormFields/Autocomplete";
 import {
@@ -10,6 +10,7 @@ import ConditionVerificationStatusMenu from "../ConditionVerificationStatusMenu"
 import { classNames, mergeQueryOptions } from "../../../Utils/utils";
 import useQuery from "../../../Utils/request/useQuery";
 import routes from "../../../Redux/api";
+import { Error } from "../../../Utils/Notifications";
 
 interface AddICD11DiagnosisProps {
   className?: string;
@@ -24,7 +25,15 @@ export default function AddICD11Diagnosis(props: AddICD11DiagnosisProps) {
   const [adding, setAdding] = useState(false);
   const hasError = !!props.disallowed.find((d) => d?.id === selected?.id);
 
-  const { data, loading, refetch } = useQuery(routes.listICD11Diagnosis);
+  const { res, data, loading, refetch } = useQuery(routes.listICD11Diagnosis, {
+    silent: true,
+  });
+
+  useEffect(() => {
+    if (res?.status === 500) {
+      Error({ msg: "ICD-11 Diagnosis functionality is facing issues." });
+    }
+  }, [res?.status]);
 
   const handleAdd = async (status: CreateDiagnosis["verification_status"]) => {
     if (!selected) return;
diff --git a/src/Components/Diagnosis/ConsultationDiagnosisBuilder/ConsultationDiagnosisEntry.tsx b/src/Components/Diagnosis/ConsultationDiagnosisBuilder/ConsultationDiagnosisEntry.tsx
index c2354503bea..d431890cd17 100644
--- a/src/Components/Diagnosis/ConsultationDiagnosisBuilder/ConsultationDiagnosisEntry.tsx
+++ b/src/Components/Diagnosis/ConsultationDiagnosisBuilder/ConsultationDiagnosisEntry.tsx
@@ -80,9 +80,11 @@ export default function ConsultationDiagnosisEntry(props: Props) {
               ? "font-semibold text-primary-500"
               : "font-normal",
             !isActive && "text-gray-500 line-through",
+            !object.diagnosis_object?.label && "italic text-gray-500",
           )}
         >
-          {object.diagnosis_object?.label}
+          {object.diagnosis_object?.label ||
+            "Unable to retrieve this ICD-11 diagnosis at the moment"}
         </span>
         <div className="flex items-center justify-end gap-2 sm:flex-row md:absolute md:inset-y-0 md:right-2 md:justify-normal">
           <div className="w-32">
diff --git a/src/Components/Diagnosis/DiagnosesListAccordion.tsx b/src/Components/Diagnosis/DiagnosesListAccordion.tsx
index 0e6d62774c7..ec641d720b5 100644
--- a/src/Components/Diagnosis/DiagnosesListAccordion.tsx
+++ b/src/Components/Diagnosis/DiagnosesListAccordion.tsx
@@ -4,7 +4,7 @@ import {
   ConsultationDiagnosis,
 } from "./types";
 import { useTranslation } from "react-i18next";
-import { compareBy } from "../../Utils/utils";
+import { classNames, compareBy } from "../../Utils/utils";
 import { useState } from "react";
 import CareIcon from "../../CAREUI/icons/CareIcon";
 import ButtonV2 from "../Common/components/ButtonV2";
@@ -96,7 +96,14 @@ const DiagnosesOfStatus = ({ diagnoses }: Props) => {
       <ul className="text-sm">
         {diagnoses.map((diagnosis) => (
           <li key={diagnosis.id} className="flex items-center gap-2">
-            <span>{diagnosis.diagnosis_object?.label}</span>
+            <span
+              className={classNames(
+                !diagnosis.diagnosis_object?.label && "italic text-gray-500",
+              )}
+            >
+              {diagnosis.diagnosis_object?.label ||
+                "Unable to resolve ICD-11 diagnosis at the moment"}
+            </span>
           </li>
         ))}
       </ul>
diff --git a/src/Components/Facility/ConsultationForm.tsx b/src/Components/Facility/ConsultationForm.tsx
index a5e333e43c6..6e1a976de35 100644
--- a/src/Components/Facility/ConsultationForm.tsx
+++ b/src/Components/Facility/ConsultationForm.tsx
@@ -1359,7 +1359,11 @@ export const ConsultationForm = ({ facilityId, patientId, id }: Props) => {
                       >
                         <FieldLabel>Procedures</FieldLabel>
                         <ProcedureBuilder
-                          procedures={state.form.procedure}
+                          procedures={
+                            Array.isArray(state.form.procedure)
+                              ? state.form.procedure
+                              : []
+                          }
                           setProcedures={(procedure) => {
                             handleFormFieldChange({
                               name: "procedure",
diff --git a/src/Components/Patient/DiagnosesFilter.tsx b/src/Components/Patient/DiagnosesFilter.tsx
index e8bd1afb722..c4c4872fdda 100644
--- a/src/Components/Patient/DiagnosesFilter.tsx
+++ b/src/Components/Patient/DiagnosesFilter.tsx
@@ -7,6 +7,7 @@ import useQuery from "../../Utils/request/useQuery";
 import routes from "../../Redux/api";
 import { mergeQueryOptions } from "../../Utils/utils";
 import { debounce } from "lodash-es";
+import { Error } from "../../Utils/Notifications";
 
 export const FILTER_BY_DIAGNOSES_KEYS = [
   "diagnoses",
@@ -34,7 +35,15 @@ interface Props {
 export default function DiagnosesFilter(props: Props) {
   const { t } = useTranslation();
   const [diagnoses, setDiagnoses] = useState<ICD11DiagnosisModel[]>([]);
-  const { data, loading, refetch } = useQuery(routes.listICD11Diagnosis);
+  const { res, data, loading, refetch } = useQuery(routes.listICD11Diagnosis, {
+    silent: true,
+  });
+
+  useEffect(() => {
+    if (res?.status === 500) {
+      Error({ msg: "ICD-11 Diagnosis functionality is facing issues." });
+    }
+  }, [res?.status]);
 
   useEffect(() => {
     if (!props.value) {

From db8b1f4cb641f6c3513ddf7790ff0e4f182c3e4b Mon Sep 17 00:00:00 2001
From: Aryan <92309448+Aryanshu919@users.noreply.github.com>
Date: Thu, 6 Jun 2024 07:54:07 +0530
Subject: [PATCH 07/11] Added padding in the user tab for "No users found"
 (#7895)

* Added padding in the user tab for No users found

* Update ManageUsers.tsx

* Update ManageUsers.tsx

* Update ManageUsers.tsx

* Update ManageUsers.tsx

* Update ManageUsers.tsx

* Update ManageUsers.tsx

* Update ManageUsers.tsx

* Update ManageUsers.tsx

* Update ManageUsers.tsx

* fix lint
---
 src/Components/Users/ManageUsers.tsx | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/Components/Users/ManageUsers.tsx b/src/Components/Users/ManageUsers.tsx
index 9669032f14c..99a88b20600 100644
--- a/src/Components/Users/ManageUsers.tsx
+++ b/src/Components/Users/ManageUsers.tsx
@@ -447,7 +447,11 @@ export default function ManageUsers() {
   } else if (userListData?.results && userListData?.results.length === 0) {
     manageUsers = (
       <div>
-        <h5> {t("no_users_found")}</h5>
+        <div className="h-full space-y-2 rounded-lg bg-white p-7 shadow">
+          <div className="flex w-full items-center justify-center text-xl font-bold text-gray-500">
+            No Users Found
+          </div>
+        </div>
       </div>
     );
   }

From 5f05c0c0d6ce15e740061e74eb85c65f8c283b04 Mon Sep 17 00:00:00 2001
From: Rithvik Nishad <mail@rithviknishad.dev>
Date: Fri, 7 Jun 2024 12:17:55 +0530
Subject: [PATCH 08/11] Show link asset popup only for IV and NIV resp. support
 types. (#7989)

---
 .../CriticalCare__VentilatorParametersEditor.res   | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/Components/CriticalCareRecording/VentilatorParametersEditor/CriticalCare__VentilatorParametersEditor.res b/src/Components/CriticalCareRecording/VentilatorParametersEditor/CriticalCare__VentilatorParametersEditor.res
index c06780aad75..0933f46bf1a 100644
--- a/src/Components/CriticalCareRecording/VentilatorParametersEditor/CriticalCare__VentilatorParametersEditor.res
+++ b/src/Components/CriticalCareRecording/VentilatorParametersEditor/CriticalCare__VentilatorParametersEditor.res
@@ -304,15 +304,11 @@ let make = (
               (state.ventilator_interface !=
                 CriticalCare__VentilatorParameters.decodeVentilatorInterfaceType(
                   ventilatorInterfaceOptions[0].value,
-                ) ||
-              switch state.bilateral_air_entry {
-              | Some(true) => true
-              | _ => false
-              } ||
-              switch (state.etco2) {
-              | Some(intValue) => true
-              | None => false
-              })
+                ) &&
+              state.ventilator_interface !=
+                CriticalCare__VentilatorParameters.decodeVentilatorInterfaceType(
+                  ventilatorInterfaceOptions[3].value,
+                ))
           ) {
             toggleOpen()
           } else {

From ed7b3079920d9d1ce7293d4bda43aebec6080940 Mon Sep 17 00:00:00 2001
From: Shivank Kacker <kacker.shivank@gmail.com>
Date: Fri, 7 Jun 2024 12:19:48 +0530
Subject: [PATCH 09/11] Restructured Consent Records (#7946)

---
 src/Common/constants.tsx                      |  15 ++
 src/Components/Common/FilePreviewDialog.tsx   |  23 +-
 src/Components/Facility/ConsultationForm.tsx  |   4 +-
 src/Components/Facility/models.tsx            |  15 +-
 .../Patient/PatientConsentRecordBlock.tsx     | 255 +++++++++---------
 .../Patient/PatientConsentRecords.tsx         | 198 +++++++-------
 src/Components/Patient/PatientInfoCard.tsx    |  46 +++-
 src/Components/Patient/models.tsx             |   1 +
 src/Redux/api.tsx                             |  25 ++
 src/Utils/useFileManager.tsx                  | 171 ++++++++++--
 src/Utils/useFileUpload.tsx                   |  25 +-
 11 files changed, 495 insertions(+), 283 deletions(-)

diff --git a/src/Common/constants.tsx b/src/Common/constants.tsx
index f0d0ab08f04..5c09b0fd201 100644
--- a/src/Common/constants.tsx
+++ b/src/Common/constants.tsx
@@ -1370,3 +1370,18 @@ export const PATIENT_NOTES_THREADS = {
 } as const;
 
 export const RATION_CARD_CATEGORY = ["BPL", "APL", "NO_CARD"] as const;
+
+export const DEFAULT_ALLOWED_EXTENSIONS = [
+  "image/*",
+  "video/*",
+  "audio/*",
+  "text/plain",
+  "text/csv",
+  "application/rtf",
+  "application/msword",
+  "application/vnd.oasis.opendocument.text",
+  "application/pdf",
+  "application/vnd.ms-excel",
+  "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
+  "application/vnd.oasis.opendocument.spreadsheet,application/pdf",
+];
diff --git a/src/Components/Common/FilePreviewDialog.tsx b/src/Components/Common/FilePreviewDialog.tsx
index 9bf9780c7a3..b4443d330ea 100644
--- a/src/Components/Common/FilePreviewDialog.tsx
+++ b/src/Components/Common/FilePreviewDialog.tsx
@@ -32,6 +32,19 @@ type FilePreviewProps = {
   fixedWidth?: boolean;
 };
 
+const previewExtensions = [
+  ".html",
+  ".htm",
+  ".pdf",
+  ".mp4",
+  ".webm",
+  ".jpg",
+  ".jpeg",
+  ".png",
+  ".gif",
+  ".webp",
+];
+
 const FilePreviewDialog = (props: FilePreviewProps) => {
   const { show, onClose, file_state, setFileState, downloadURL, fileUrl } =
     props;
@@ -130,13 +143,21 @@ const FilePreviewDialog = (props: FilePreviewProps) => {
                   }}
                   pageNumber={page}
                 />
-              ) : (
+              ) : previewExtensions.includes(file_state.extension) ? (
                 <iframe
                   sandbox=""
                   title="Source Files"
                   src={fileUrl}
                   className="h-[75vh] w-full"
                 />
+              ) : (
+                <div className="flex h-full w-full flex-col items-center justify-center">
+                  <CareIcon
+                    icon="l-file"
+                    className="mb-4 text-5xl text-gray-600"
+                  />
+                  Can't preview this file. Try downloading it.
+                </div>
               )}
             </div>
           </div>
diff --git a/src/Components/Facility/ConsultationForm.tsx b/src/Components/Facility/ConsultationForm.tsx
index 6e1a976de35..a728ffc805a 100644
--- a/src/Components/Facility/ConsultationForm.tsx
+++ b/src/Components/Facility/ConsultationForm.tsx
@@ -1,6 +1,6 @@
 import * as Notification from "../../Utils/Notifications.js";
 
-import { BedModel, ConsentRecord, FacilityModel } from "./models";
+import { BedModel, FacilityModel } from "./models";
 import {
   CONSULTATION_SUGGESTION,
   DISCHARGE_REASONS,
@@ -119,7 +119,6 @@ type FormDetails = {
   death_confirmed_doctor: string;
   InvestigationAdvice: InvestigationType[];
   procedures: ProcedureType[];
-  consent_records: ConsentRecord[];
 };
 
 const initForm: FormDetails = {
@@ -170,7 +169,6 @@ const initForm: FormDetails = {
   death_confirmed_doctor: "",
   InvestigationAdvice: [],
   procedures: [],
-  consent_records: [],
 };
 
 const initError = Object.assign(
diff --git a/src/Components/Facility/models.tsx b/src/Components/Facility/models.tsx
index 8e54f0896bc..74a0d4a0f26 100644
--- a/src/Components/Facility/models.tsx
+++ b/src/Components/Facility/models.tsx
@@ -99,12 +99,18 @@ export interface OptionsType {
 
 export type PatientCategory = "Comfort Care" | "Mild" | "Moderate" | "Critical";
 
-export type ConsentRecord = {
+export interface PatientConsentModel {
   id: string;
   type: (typeof CONSENT_TYPE_CHOICES)[number]["id"];
-  patient_code_status?: (typeof CONSENT_PATIENT_CODE_STATUS_CHOICES)[number]["id"];
-  deleted?: boolean;
-};
+  patient_code_status:
+    | (typeof CONSENT_PATIENT_CODE_STATUS_CHOICES)[number]["id"]
+    | null;
+  archived: boolean;
+  archived_by?: UserBareMinimum;
+  archived_date: string;
+  created_date: string;
+  created_by: UserBareMinimum;
+}
 
 export interface ConsultationModel {
   encounter_date: string;
@@ -174,7 +180,6 @@ export interface ConsultationModel {
   is_readmission?: boolean;
   medico_legal_case?: boolean;
   investigation?: InvestigationType[];
-  consent_records?: ConsentRecord[];
 }
 
 export interface PatientStatsModel {
diff --git a/src/Components/Patient/PatientConsentRecordBlock.tsx b/src/Components/Patient/PatientConsentRecordBlock.tsx
index 9c1969ae5a3..4998ded0acf 100644
--- a/src/Components/Patient/PatientConsentRecordBlock.tsx
+++ b/src/Components/Patient/PatientConsentRecordBlock.tsx
@@ -3,185 +3,174 @@ import {
   CONSENT_PATIENT_CODE_STATUS_CHOICES,
   CONSENT_TYPE_CHOICES,
 } from "../../Common/constants";
-import routes from "../../Redux/api";
-import useQuery from "../../Utils/request/useQuery";
-import { ConsentRecord } from "../Facility/models";
 import { FileUploadModel } from "./models";
 import CareIcon from "../../CAREUI/icons/CareIcon";
 import ButtonV2 from "../Common/components/ButtonV2";
-import { useEffect } from "react";
 import useAuthUser from "../../Common/hooks/useAuthUser";
+import { PatientConsentModel } from "../Facility/models";
+import { SelectFormField } from "../Form/FormFields/SelectFormField";
+import { useEffect, useState } from "react";
+import request from "../../Utils/request/request";
+import routes from "../../Redux/api";
 
 export default function PatientConsentRecordBlockGroup(props: {
-  consentRecord: ConsentRecord;
+  consentRecord: PatientConsentModel;
+  consultationId: string;
   previewFile: (file: FileUploadModel, file_associating_id: string) => void;
   archiveFile: (
     file: FileUploadModel,
     file_associating_id: string,
     skipPrompt?: { reason: string },
   ) => void;
-  onDelete: (consentRecord: ConsentRecord) => void;
-  refreshTrigger: any;
+  editFile: (file: FileUploadModel) => void;
   showArchive: boolean;
-  onFilesFound: () => void;
+  files?: FileUploadModel[];
 }) {
   const {
     consentRecord,
     previewFile,
     archiveFile,
-    refreshTrigger,
+    editFile,
+    files,
     showArchive,
+    consultationId,
   } = props;
 
   const authUser = useAuthUser();
 
-  const filesQuery = useQuery(routes.viewUpload, {
-    query: {
-      file_type: "CONSENT_RECORD",
-      associating_id: consentRecord.id,
-      is_archived: false,
-      limit: 100,
-      offset: 0,
-    },
-    onResponse: (response) => {
-      /*
-      if (consentRecord.deleted === true && response.data?.results) {
-        const unarchivedFiles = response.data.results;
-        console.log("checking for unarchived files on this deleted consent record")
-        for (const file of unarchivedFiles) {
-          console.log("archiving file", file)
-          archiveFile(file, consentRecord.id, {
-            reason: "Consent Record Archived",
-          });
-        }
-      }
-      */
-
-      if ((response.data?.results?.length || 0) > 0) {
-        props.onFilesFound();
-      }
-    },
-  });
-
-  const archivedFilesQuery = useQuery(routes.viewUpload, {
-    query: {
-      file_type: "CONSENT_RECORD",
-      associating_id: consentRecord.id,
-      is_archived: true,
-      limit: 100,
-      offset: 0,
-    },
-    prefetch: showArchive,
-    onResponse: (response) => {
-      if ((response.data?.results?.length || 0) > 0) {
-        props.onFilesFound();
-      }
-    },
-  });
-
   const consent = CONSENT_TYPE_CHOICES.find((c) => c.id === consentRecord.type);
   const consentPCS = CONSENT_PATIENT_CODE_STATUS_CHOICES.find(
     (c) => c.id === consentRecord.patient_code_status,
   );
+  const [patientCodeStatus, setPatientCodeStatus] = useState(1);
 
-  const data = showArchive
-    ? [
-        ...(archivedFilesQuery.data?.results || []),
-        ...(consentRecord.deleted ? filesQuery.data?.results || [] : []),
-      ]
-    : filesQuery.data?.results;
-
-  const loading = archivedFilesQuery.loading || filesQuery.loading;
+  const handlePCSUpdate = async (status: number) => {
+    const res = await request(routes.partialUpdateConsent, {
+      pathParams: {
+        id: consentRecord.id,
+        consultationId: consultationId,
+      },
+      body: {
+        patient_code_status: status,
+      },
+    });
+    if (res.data) window.location.reload();
+  };
 
   useEffect(() => {
-    if (!showArchive) {
-      filesQuery.refetch();
-    } else {
-      archivedFilesQuery.refetch();
-    }
-  }, [showArchive, refreshTrigger]);
+    if (consentRecord.patient_code_status !== null)
+      setPatientCodeStatus(consentRecord.patient_code_status);
+  }, [consentRecord]);
+
+  // see if the user has permission to edit the file.
+  // only the user who uploaded the file, district admin and state admin can edit the file
+  const hasEditPermission = (file: FileUploadModel) =>
+    file?.uploaded_by?.username === authUser.username ||
+    authUser.user_type === "DistrictAdmin" ||
+    authUser.user_type === "StateAdmin";
 
   return (
     <div
-      className={`flex flex-col gap-2 ${(data?.length || 0) < 1 && "hidden"}`}
+      className={`flex flex-col gap-2 ${(files?.length || 0) < 1 && "hidden"}`}
     >
       <div className="flex items-center justify-between">
         <div>
           <h4>
             {consent?.text} {consentPCS?.text && `(${consentPCS.text})`}
           </h4>
-          {consentRecord.deleted && (
-            <div>
-              <div className="text-sm">
-                <CareIcon icon="l-archive" className="mr-1" />
-                Archived
-              </div>
-            </div>
-          )}
         </div>
-        {/*
-        {!consentRecord.deleted && !showArchive && (
-          <button
-            className="text-red-500 hover:text-red-600"
-            onClick={() => props.onDelete(consentRecord)}
-          >
-            <CareIcon icon="l-archive" className="mr-1" />
-            Archive
-          </button>
-        )}
-      */}
       </div>
-      {loading ? (
-        <div className="skeleton-animate-alpha h-32 rounded-lg" />
-      ) : (
-        data?.map((file: FileUploadModel, i: number) => (
-          <div
-            key={i}
-            className={`flex flex-col justify-between gap-2 rounded-lg border border-gray-300 md:flex-row md:items-center ${showArchive ? "text-gray-600" : "bg-white"}  px-4 py-2 transition-all hover:bg-gray-100`}
+      {consentRecord.type === 2 && consentRecord.patient_code_status === 0 && (
+        <div className="flex gap-2">
+          <SelectFormField
+            name="patient_code_status"
+            className="flex-1"
+            onChange={(e) => {
+              console.log(e.value);
+              setPatientCodeStatus(e.value);
+            }}
+            value={
+              CONSENT_PATIENT_CODE_STATUS_CHOICES.find(
+                (c) => c.id === patientCodeStatus,
+              )?.id
+            }
+            optionValue={(option: any) => option.id}
+            optionLabel={(option: any) => option.text}
+            options={CONSENT_PATIENT_CODE_STATUS_CHOICES}
+            required
+            error="Please select a patient code status type"
+          />
+          <ButtonV2
+            onClick={() => {
+              handlePCSUpdate(patientCodeStatus);
+            }}
+            disabled={patientCodeStatus === consentRecord.patient_code_status}
+            className="h-[46px]"
           >
-            <div className="flex items-center gap-4 ">
-              <div>
-                <CareIcon icon="l-file" className="text-5xl text-gray-600" />
+            Update
+          </ButtonV2>
+        </div>
+      )}
+      {files?.map((file: FileUploadModel, i: number) => (
+        <div
+          key={i}
+          className={`flex flex-col justify-between gap-2 rounded-lg border border-gray-300 xl:flex-row xl:items-center ${showArchive ? "text-gray-600" : "bg-white"}  px-4 py-2 transition-all hover:bg-gray-100`}
+        >
+          <div className="flex items-center gap-4 ">
+            <div>
+              <CareIcon icon="l-file" className="text-5xl text-gray-600" />
+            </div>
+            <div className="min-w-[40%] break-all">
+              <div className="">
+                {file.name}
+                {file.extension} {file.is_archived && "(Archived)"}
               </div>
-              <div className="min-w-[40%] break-all">
-                <div className="">
-                  {file.name}
-                  {file.extension} {file.is_archived && "(Archived)"}
-                </div>
-                <div className="text-xs text-gray-700">
-                  {dayjs(file.created_date).format("DD MMM YYYY, hh:mm A")}
-                </div>
+              <div className="text-xs text-gray-700">
+                {dayjs(
+                  file.is_archived ? file.archived_datetime : file.created_date,
+                ).format("DD MMM YYYY, hh:mm A")}{" "}
+                by{" "}
+                {file.is_archived
+                  ? file.archived_by?.username
+                  : file.uploaded_by?.username}
               </div>
             </div>
-            <div className="flex shrink-0 justify-end gap-2">
-              {!file.is_archived && (
-                <ButtonV2
-                  onClick={() => previewFile(file, consentRecord.id)}
-                  className=""
-                >
-                  <CareIcon icon="l-eye" />
-                  View
-                </ButtonV2>
-              )}
-              {(file.is_archived ||
-                file?.uploaded_by?.username === authUser.username ||
-                authUser.user_type === "DistrictAdmin" ||
-                authUser.user_type === "StateAdmin") && (
-                <ButtonV2
-                  variant={file.is_archived ? "primary" : "secondary"}
-                  onClick={() => archiveFile(file, consentRecord.id)}
-                  className=""
-                >
-                  <CareIcon
-                    icon={file.is_archived ? "l-info-circle" : "l-archive"}
-                  />
-                  {file.is_archived ? "More Info" : "Archive"}
-                </ButtonV2>
-              )}
-            </div>
           </div>
-        ))
-      )}
+          <div className="flex shrink-0 flex-wrap justify-end gap-2">
+            {!file.is_archived && (
+              <ButtonV2
+                onClick={() => previewFile(file, consentRecord.id)}
+                className=""
+              >
+                <CareIcon icon="l-eye" />
+                View
+              </ButtonV2>
+            )}
+            {!file.is_archived && hasEditPermission(file) && (
+              <ButtonV2
+                variant={"secondary"}
+                onClick={() => editFile(file)}
+                className=""
+              >
+                <CareIcon icon={"l-pen"} />
+                Rename
+              </ButtonV2>
+            )}
+            {(file.is_archived || hasEditPermission(file)) && (
+              <ButtonV2
+                variant={file.is_archived ? "primary" : "secondary"}
+                onClick={() => archiveFile(file, consentRecord.id)}
+                className=""
+              >
+                <CareIcon
+                  icon={file.is_archived ? "l-info-circle" : "l-archive"}
+                />
+                {file.is_archived ? "More Info" : "Archive"}
+              </ButtonV2>
+            )}
+          </div>
+        </div>
+      ))}
     </div>
   );
 }
diff --git a/src/Components/Patient/PatientConsentRecords.tsx b/src/Components/Patient/PatientConsentRecords.tsx
index ca26b270d80..d850179c6d0 100644
--- a/src/Components/Patient/PatientConsentRecords.tsx
+++ b/src/Components/Patient/PatientConsentRecords.tsx
@@ -6,7 +6,6 @@ import {
 import routes from "../../Redux/api";
 import useQuery from "../../Utils/request/useQuery";
 import Page from "../Common/components/Page";
-import { ConsentRecord } from "../Facility/models";
 import request from "../../Utils/request/request";
 import ConfirmDialog from "../Common/ConfirmDialog";
 import { SelectFormField } from "../Form/FormFields/SelectFormField";
@@ -18,6 +17,7 @@ import useFileUpload from "../../Utils/useFileUpload";
 import PatientConsentRecordBlockGroup from "./PatientConsentRecordBlock";
 import SwitchTabs from "../Common/components/SwitchTabs";
 import useFileManager from "../../Utils/useFileManager";
+import { PatientConsentModel } from "../Facility/models";
 
 export default function PatientConsentRecords(props: {
   facilityId: string;
@@ -26,7 +26,6 @@ export default function PatientConsentRecords(props: {
 }) {
   const { facilityId, patientId, consultationId } = props;
   const [showArchived, setShowArchived] = useState(false);
-  const [filesFound, setFilesFound] = useState(false);
   const [showPCSChangeModal, setShowPCSChangeModal] = useState<number | null>(
     null,
   );
@@ -35,14 +34,24 @@ export default function PatientConsentRecords(props: {
     patient_code_status: 4,
   });
 
+  const refetchAll = () => {
+    refetch();
+    refetchFiles();
+    refetchArchivedFiles();
+  };
+
   const fileUpload = useFileUpload({
     type: "CONSENT_RECORD",
+    allowedExtensions: ["pdf", "jpg", "jpeg", "png"],
   });
 
   const fileManager = useFileManager({
     type: "CONSENT_RECORD",
     onArchive: async () => {
-      refetch();
+      refetchAll();
+    },
+    onEdit: async () => {
+      refetchAll();
     },
   });
 
@@ -51,36 +60,48 @@ export default function PatientConsentRecords(props: {
       id: patientId,
     },
   });
-  const {
-    data: consultation,
-    refetch,
-    loading,
-  } = useQuery(routes.getConsultation, {
-    pathParams: { id: consultationId! },
-    onResponse: (data) => {
-      if (data.data && data.data.consent_records) {
-        setConsentRecords(data.data.consent_records);
-      }
+
+  const { data: consentRecordsData, refetch } = useQuery(routes.listConsents, {
+    pathParams: {
+      consultationId,
+    },
+    query: {
+      limit: 1000,
+      offset: 0,
     },
   });
 
-  const [showDeleteConsent, setShowDeleteConsent] = useState<string | null>(
-    null,
+  const consentRecords = consentRecordsData?.results;
+
+  const { data: unarchivedFiles, refetch: refetchFiles } = useQuery(
+    routes.viewUpload,
+    {
+      query: {
+        file_type: "CONSENT_RECORD",
+        associating_id: consentRecords?.map((cr) => cr.id).join(","),
+        limit: 1000,
+        offset: 0,
+        is_archived: false,
+      },
+      prefetch: (consentRecords?.length || 0) > 0 && showArchived,
+    },
   );
 
-  const [consentRecords, setConsentRecords] = useState<ConsentRecord[] | null>(
-    null,
+  const { data: archivedFiles, refetch: refetchArchivedFiles } = useQuery(
+    routes.viewUpload,
+    {
+      query: {
+        file_type: "CONSENT_RECORD",
+        associating_id: consentRecords?.map((cr) => cr.id).join(","),
+        limit: 1000,
+        offset: 0,
+        is_archived: true,
+      },
+      prefetch: (consentRecords?.length || 0) > 0 && !showArchived,
+    },
   );
 
-  const handleDeleteConsent = async () => {
-    const consent_id = showDeleteConsent;
-    if (!consent_id || !consultationId || !consentRecords) return;
-    const newRecords = consentRecords.map((cr) =>
-      cr.id === consent_id ? { ...cr, deleted: true } : cr,
-    );
-    setConsentRecords(newRecords);
-    setShowDeleteConsent(null);
-  };
+  const files = showArchived ? archivedFiles : unarchivedFiles;
 
   const selectField = (name: string) => {
     return {
@@ -91,55 +112,36 @@ export default function PatientConsentRecords(props: {
     };
   };
 
-  const handleUpload = async (diffPCS?: ConsentRecord) => {
-    if (newConsent.type === 0) return;
-    const consentTypeExists = consentRecords?.find(
-      (record) => record.type === newConsent.type && record.deleted !== true,
+  const handleUpload = async (diffPCS?: PatientConsentModel) => {
+    const consentExists = consentRecords?.find(
+      (record) => record.type === newConsent.type && !record.archived,
     );
-    if (consentTypeExists && !diffPCS) {
-      await fileUpload.handleFileUpload(consentTypeExists.id);
-    } else {
-      const randomId = "consent-" + new Date().getTime().toString();
-      const newRecords = [
-        ...(consentRecords?.map((r) =>
-          r.id === diffPCS?.id ? { ...r, deleted: true } : r,
-        ) || []),
-        {
-          id: randomId,
-          type: newConsent.type,
+    let consentId = consentExists?.id;
+    if (!consentExists || diffPCS) {
+      consentId = undefined;
+      const res = await request(routes.createConsent, {
+        pathParams: { consultationId: consultationId },
+        body: {
+          ...newConsent,
           patient_code_status:
             newConsent.type === 2 ? newConsent.patient_code_status : undefined,
         },
-      ];
-      await request(routes.partialUpdateConsultation, {
-        pathParams: { id: consultationId },
-        body: { consent_records: newRecords },
       });
-      await fileUpload.handleFileUpload(randomId);
-      setConsentRecords(newRecords);
+      if (res.data) {
+        consentId = res.data.id;
+      }
     }
-
+    consentId && (await fileUpload.handleFileUpload(consentId));
     refetch();
   };
 
   useEffect(() => {
-    const timeout = setTimeout(async () => {
-      if (consentRecords) {
-        await request(routes.partialUpdateConsultation, {
-          pathParams: { id: consultationId },
-          body: { consent_records: consentRecords },
-        });
-      }
-    }, 1000);
-    return () => clearTimeout(timeout);
+    if (consentRecords && consentRecords.length > 0) {
+      refetchFiles();
+      refetchArchivedFiles();
+    }
   }, [consentRecords]);
 
-  const tabConsents = consentRecords?.filter((c) => showArchived || !c.deleted);
-
-  useEffect(() => {
-    setFilesFound(false);
-  }, [showArchived]);
-
   return (
     <Page
       title={"Patient Consent Records"}
@@ -159,18 +161,6 @@ export default function PatientConsentRecords(props: {
     >
       {fileUpload.Dialogues}
       {fileManager.Dialogues}
-      <ConfirmDialog
-        show={showDeleteConsent !== null}
-        onClose={() => setShowDeleteConsent(null)}
-        onConfirm={handleDeleteConsent}
-        action="Archive"
-        variant="danger"
-        description={
-          "Are you sure you want to archive this consent record? You can find it in the archive section."
-        }
-        title="Archive Consent"
-        className="w-auto"
-      />
       <ConfirmDialog
         show={showPCSChangeModal !== null}
         onClose={() => setShowPCSChangeModal(null)}
@@ -180,7 +170,7 @@ export default function PatientConsentRecords(props: {
               consentRecords?.find(
                 (record) =>
                   record.type === 2 &&
-                  !record.deleted &&
+                  !record.archived &&
                   record.patient_code_status !== showPCSChangeModal,
               ),
             );
@@ -189,7 +179,7 @@ export default function PatientConsentRecords(props: {
         }}
         action="Change Patient Code Status"
         variant="danger"
-        description={`Consent records exist with the "${CONSENT_PATIENT_CODE_STATUS_CHOICES.find((c) => consentRecords?.find((c) => c.type === 2 && !c.deleted)?.patient_code_status === c.id)?.text}" patient code status. Adding a new record for a different type will archive the existing records. Are you sure you want to proceed?`}
+        description={`Consent records exist with the "${CONSENT_PATIENT_CODE_STATUS_CHOICES.find((c) => consentRecords?.find((c) => c.type === 2 && !c.archived)?.patient_code_status === c.id)?.text}" patient code status. Adding a new record for a different type will archive the existing records. Are you sure you want to proceed?`}
         title="Archive Previous Records"
         className="w-auto"
       />
@@ -246,7 +236,7 @@ export default function PatientConsentRecords(props: {
                         newConsent.type === 2 &&
                         record.patient_code_status !==
                           newConsent.patient_code_status &&
-                        record.deleted !== true,
+                        record.archived !== true,
                     );
                     if (diffPCS) {
                       setShowPCSChangeModal(newConsent.patient_code_status);
@@ -255,6 +245,10 @@ export default function PatientConsentRecords(props: {
                     }
                   }}
                   loading={!!fileUpload.progress}
+                  disabled={
+                    newConsent.type === 2 &&
+                    newConsent.patient_code_status === 0
+                  }
                   className="flex-1"
                 >
                   <CareIcon icon="l-check" className="mr-2" />
@@ -281,29 +275,37 @@ export default function PatientConsentRecords(props: {
               </>
             )}
           </div>
+          <div className="mt-2 text-sm text-red-500">{fileUpload.error}</div>
         </div>
         <div className="flex-1">
-          <div className="flex flex-col gap-4">
-            {loading ? (
+          {consentRecords?.filter(
+            (r) =>
+              files?.results.filter((f) => f.associating_id === r.id).length,
+          ).length === 0 ? (
+            <div className="flex h-32 items-center justify-center text-gray-500">
+              No consent records found
+            </div>
+          ) : (
+            (!unarchivedFiles || !archivedFiles) &&
+            !consentRecords && (
               <div className="skeleton-animate-alpha h-32 rounded-lg" />
-            ) : tabConsents?.length === 0 || !filesFound ? (
-              <div className="flex h-32 items-center justify-center text-gray-500">
-                No records found
-              </div>
-            ) : null}
-            {!loading &&
-              tabConsents?.map((record, index) => (
-                <PatientConsentRecordBlockGroup
-                  key={index}
-                  consentRecord={record}
-                  previewFile={fileManager.viewFile}
-                  archiveFile={fileManager.archiveFile}
-                  onDelete={(record) => setShowDeleteConsent(record.id)}
-                  refreshTrigger={consultation}
-                  showArchive={showArchived}
-                  onFilesFound={() => setFilesFound(true)}
-                />
-              ))}
+            )
+          )}
+          <div className="flex flex-col gap-4">
+            {consentRecords?.map((record, index) => (
+              <PatientConsentRecordBlockGroup
+                key={index}
+                consultationId={consultationId}
+                consentRecord={record}
+                previewFile={fileManager.viewFile}
+                archiveFile={fileManager.archiveFile}
+                editFile={fileManager.editFile}
+                showArchive={showArchived}
+                files={files?.results.filter(
+                  (f) => f.associating_id === record.id,
+                )}
+              />
+            ))}
           </div>
         </div>
       </div>
diff --git a/src/Components/Patient/PatientInfoCard.tsx b/src/Components/Patient/PatientInfoCard.tsx
index 247d9312d6e..0b05009d4a2 100644
--- a/src/Components/Patient/PatientInfoCard.tsx
+++ b/src/Components/Patient/PatientInfoCard.tsx
@@ -140,6 +140,30 @@ export default function PatientInfoCard(props: {
     prefetch: !!consultation?.treating_physician_object?.username,
   });
 
+  const { data: consentRecords, loading: consentRecordsLoading } = useQuery(
+    routes.listConsents,
+    {
+      pathParams: {
+        consultationId: consultation?.id ?? "",
+      },
+      prefetch: !!consultation?.id,
+    },
+  );
+
+  const { data: consentFiles, loading: consentFilesLoading } = useQuery(
+    routes.viewUpload,
+    {
+      query: {
+        file_type: "CONSENT_RECORD",
+        associating_id: consentRecords?.results.map((cr) => cr.id).join(","),
+        limit: 1,
+        offset: 0,
+        is_archived: false,
+      },
+      prefetch: (consentRecords?.results.length || 0) > 0,
+    },
+  );
+
   return (
     <>
       <DialogModal
@@ -354,18 +378,18 @@ export default function PatientInfoCard(props: {
                         </div>
                       </div>
                     )}
-                  {(
-                    consultation?.consent_records?.filter((c) => !c.deleted) ||
-                    []
-                  ).length < 1 && (
-                    <div>
-                      <div className="inline-flex w-full items-center justify-start rounded border border-gray-500 bg-red-400 p-1 px-3 text-xs font-semibold leading-4">
-                        <span className="font-semibold text-white">
-                          Consent Records Missing
-                        </span>
+                  {!consentFilesLoading &&
+                    !consentRecordsLoading &&
+                    !consentFiles?.results.filter((c) => !c.is_archived)
+                      .length && (
+                      <div>
+                        <div className="inline-flex w-full items-center justify-start rounded border border-red-600 bg-red-400 p-1 px-3 text-xs font-semibold leading-4">
+                          <span className="font-semibold text-white">
+                            Consent Records Missing
+                          </span>
+                        </div>
                       </div>
-                    </div>
-                  )}
+                    )}
                   {consultation?.suggestion === "DC" && (
                     <div>
                       <div>
diff --git a/src/Components/Patient/models.tsx b/src/Components/Patient/models.tsx
index f64d71146c4..9664d0f462b 100644
--- a/src/Components/Patient/models.tsx
+++ b/src/Components/Patient/models.tsx
@@ -354,6 +354,7 @@ export interface CreateFileResponse {
 export interface FileUploadModel {
   id?: string;
   name?: string;
+  associating_id?: string;
   created_date?: string;
   upload_completed?: boolean;
   uploaded_by?: PerformedByModel;
diff --git a/src/Redux/api.tsx b/src/Redux/api.tsx
index 439b122ca09..8cda2038ecc 100644
--- a/src/Redux/api.tsx
+++ b/src/Redux/api.tsx
@@ -65,6 +65,7 @@ import {
   LocalBodyModel,
   LocationModel,
   MinimumQuantityItemResponse,
+  PatientConsentModel,
   PatientNotesEditModel,
   PatientNotesModel,
   PatientStatsModel,
@@ -1000,6 +1001,30 @@ const routes = {
     TRes: Type<PaginatedResponse<PatientModel>>(),
   },
 
+  // Consents
+  listConsents: {
+    path: "/api/v1/consultation/{consultationId}/consents/",
+    method: "GET",
+    TRes: Type<PaginatedResponse<PatientConsentModel>>(),
+  },
+  getConsent: {
+    path: "/api/v1/consultation/{consultationId}/consents/{id}/",
+    method: "GET",
+    TRes: Type<PatientConsentModel>(),
+  },
+  createConsent: {
+    path: "/api/v1/consultation/{consultationId}/consents/",
+    method: "POST",
+    TRes: Type<PatientConsentModel>(),
+    TBody: Type<Partial<PatientConsentModel>>(),
+  },
+  partialUpdateConsent: {
+    path: "/api/v1/consultation/{consultationId}/consents/{id}/",
+    method: "PATCH",
+    TRes: Type<PatientConsentModel>(),
+    TBody: Type<Partial<PatientConsentModel>>(),
+  },
+
   //Profile
   checkUsername: {
     path: "/api/v1/users/{username}/check_availability/",
diff --git a/src/Utils/useFileManager.tsx b/src/Utils/useFileManager.tsx
index 133b4d2533d..d45ddc28f06 100644
--- a/src/Utils/useFileManager.tsx
+++ b/src/Utils/useFileManager.tsx
@@ -10,10 +10,12 @@ import TextAreaFormField from "../Components/Form/FormFields/TextAreaFormField";
 import { Cancel, Submit } from "../Components/Common/components/ButtonV2";
 import { formatDateTime } from "./utils";
 import * as Notification from "./Notifications.js";
+import TextFormField from "../Components/Form/FormFields/TextFormField";
 
 export interface FileManagerOptions {
   type: string;
   onArchive?: () => void;
+  onEdit?: () => void;
 }
 
 export interface FileManagerResult {
@@ -23,13 +25,14 @@ export interface FileManagerResult {
     associating_id: string,
     skipPrompt?: { reason: string },
   ) => void;
+  editFile: (file: FileUploadModel) => void;
   Dialogues: React.ReactNode;
 }
 
 export default function useFileManager(
   options: FileManagerOptions,
 ): FileManagerResult {
-  const { type: fileType, onArchive } = options;
+  const { type: fileType, onArchive, onEdit } = options;
 
   const [file_state, setFileState] = useState<StateInterface>({
     open: false,
@@ -49,6 +52,10 @@ export default function useFileManager(
   const [archiveReason, setArchiveReason] = useState("");
   const [archiveReasonError, setArchiveReasonError] = useState("");
   const [archiving, setArchiving] = useState(false);
+  const [editing, setEditing] = useState(false);
+  const [editDialogueOpen, setEditDialogueOpen] =
+    useState<FileUploadModel | null>(null);
+  const [editError, setEditError] = useState("");
 
   const getExtension = (url: string) => {
     const div1 = url.split("?")[0].split(".");
@@ -155,6 +162,43 @@ export default function useFileManager(
     });
   };
 
+  const validateEditFileName = (name: string) => {
+    if (name.trim() === "") {
+      setEditError("Please enter a name!");
+      return false;
+    } else {
+      setEditError("");
+      return true;
+    }
+  };
+
+  const partialupdateFileName = async (file: FileUploadModel) => {
+    if (!validateEditFileName(file.name || "")) {
+      setEditing(false);
+      return;
+    }
+
+    const { res } = await request(routes.editUpload, {
+      body: { name: file.name },
+      pathParams: {
+        id: file.id || "",
+        fileType,
+        associatingId: file.associating_id || "",
+      },
+    });
+
+    if (res?.ok) {
+      Notification.Success({ msg: "File name changed successfully" });
+      setEditDialogueOpen(null);
+      onEdit && onEdit();
+    }
+    setEditing(false);
+  };
+
+  const editFile = (file: FileUploadModel) => {
+    setEditDialogueOpen(file);
+  };
+
   const Dialogues = (
     <>
       <FilePreviewDialog
@@ -224,44 +268,120 @@ export default function useFileManager(
           archiveDialogueOpen !== null &&
           archiveDialogueOpen.archived_datetime !== null
         }
+        title={
+          <h1 className="text-xl text-black">
+            {archiveDialogueOpen?.name} (Archived)
+          </h1>
+        }
+        fixedWidth={false}
+        className="md:w-[700px]"
+        onClose={() => setArchiveDialogueOpen(null)}
+      >
+        <div className="mb-8 text-xs text-gray-700">
+          <CareIcon icon="l-archive" className="mr-2" />
+          This file has been archived and cannot be unarchived.
+        </div>
+        <div className="flex flex-col gap-4 md:grid md:grid-cols-2">
+          {[
+            {
+              label: "File Name",
+              content: archiveDialogueOpen?.name,
+              icon: "l-file",
+            },
+            {
+              label: "Uploaded By",
+              content: archiveDialogueOpen?.uploaded_by?.username,
+              icon: "l-user",
+            },
+            {
+              label: "Uploaded On",
+              content: formatDateTime(archiveDialogueOpen?.created_date),
+              icon: "l-clock",
+            },
+            {
+              label: "Archive Reason",
+              content: archiveDialogueOpen?.archive_reason,
+              icon: "l-archive",
+            },
+            {
+              label: "Archived By",
+              content: archiveDialogueOpen?.archived_by?.username,
+              icon: "l-user",
+            },
+            {
+              label: "Archived On",
+              content: formatDateTime(archiveDialogueOpen?.archived_datetime),
+              icon: "l-clock",
+            },
+          ].map((item, index) => (
+            <div key={index} className="flex gap-2">
+              <div className="flex aspect-square h-10 items-center justify-center rounded-full bg-primary-100">
+                <CareIcon
+                  icon={item.icon as any}
+                  className="text-lg text-primary-500"
+                />
+              </div>
+              <div>
+                <div className="text-xs uppercase text-gray-700">
+                  {item.label}
+                </div>
+                <div className="break-words text-base">{item.content}</div>
+              </div>
+            </div>
+          ))}
+        </div>
+        <div className="mt-10 flex justify-end">
+          <Cancel onClick={(_) => setArchiveDialogueOpen(null)} />
+        </div>
+      </DialogModal>
+      <DialogModal
+        show={editDialogueOpen !== null}
         title={
           <div className="flex flex-row">
-            <div className="my-1 mr-3 rounded-full bg-primary-100 px-5 py-4 text-center">
+            <div className="rounded-full bg-primary-100 px-5 py-4">
               <CareIcon
-                icon="l-question-circle"
+                icon="l-edit-alt"
                 className="text-lg text-primary-500"
               />
             </div>
-            <div className="text-grey-200 text-sm">
-              <h1 className="text-xl text-black">File Details</h1>
-              This file is archived. Once a file is archived it cannot be
-              unarchived.
+            <div className="m-4">
+              <h1 className="text-xl text-black ">Rename File</h1>
             </div>
           </div>
         }
-        onClose={() => setArchiveDialogueOpen(null)}
+        onClose={() => setEditDialogueOpen(null)}
       >
-        <div className="flex flex-col">
+        <form
+          onSubmit={(event: any) => {
+            event.preventDefault();
+            setEditing(true);
+            if (editDialogueOpen) partialupdateFileName(editDialogueOpen);
+          }}
+          className="flex w-full flex-col"
+        >
           <div>
-            <div className="text-md m-2 text-center">
-              <b id="archive-file-name">{archiveDialogueOpen?.name}</b> file is
-              archived.
-            </div>
-            <div className="text-md text-center" id="archive-file-reason">
-              <b>Reason:</b> {archiveDialogueOpen?.archive_reason}
-            </div>
-            <div className="text-md text-center">
-              <b>Archived by:</b> {archiveDialogueOpen?.archived_by?.username}
-            </div>
-            <div className="text-md text-center">
-              <b>Time of Archive: </b>
-              {formatDateTime(archiveDialogueOpen?.archived_datetime)}
-            </div>
+            <TextFormField
+              name="editFileName"
+              label="Enter the file name"
+              value={editDialogueOpen?.name}
+              onChange={(e) => {
+                setEditDialogueOpen({ ...editDialogueOpen, name: e.value });
+              }}
+              error={editError}
+            />
           </div>
           <div className="mt-4 flex flex-col-reverse justify-end gap-2 md:flex-row">
-            <Cancel onClick={(_) => setArchiveDialogueOpen(null)} />
+            <Cancel onClick={() => setEditDialogueOpen(null)} />
+            <Submit
+              disabled={
+                editing === true ||
+                editDialogueOpen?.name === "" ||
+                editDialogueOpen?.name?.length === 0
+              }
+              label="Proceed"
+            />
           </div>
-        </div>
+        </form>
       </DialogModal>
     </>
   );
@@ -269,6 +389,7 @@ export default function useFileManager(
   return {
     viewFile,
     archiveFile,
+    editFile,
     Dialogues,
   };
 }
diff --git a/src/Utils/useFileUpload.tsx b/src/Utils/useFileUpload.tsx
index e28e8c9b1b5..f072d66d6bc 100644
--- a/src/Utils/useFileUpload.tsx
+++ b/src/Utils/useFileUpload.tsx
@@ -16,6 +16,7 @@ import routes from "../Redux/api";
 import uploadFile from "./request/uploadFile";
 import * as Notification from "./Notifications.js";
 import imageCompression from "browser-image-compression";
+import { DEFAULT_ALLOWED_EXTENSIONS } from "../Common/constants";
 
 export type FileUploadOptions = {
   type: string;
@@ -23,10 +24,10 @@ export type FileUploadOptions = {
   onUpload?: (file: FileUploadModel) => void;
 } & (
   | {
-      allowAllExtensions?: boolean;
+      allowedExtensions?: string[];
     }
   | {
-      allowedExtensions?: string[];
+      allowAllExtensions?: boolean;
     }
 );
 
@@ -146,6 +147,16 @@ export default function useFileUpload(
       setError("Maximum size of files is 100 MB");
       return false;
     }
+    const extension = f.name.split(".").pop();
+    if (
+      "allowedExtensions" in options &&
+      !options.allowedExtensions?.includes(extension || "")
+    ) {
+      setError(
+        `Invalid file type ".${extension}" Allowed types: ${options.allowedExtensions?.join(", ")}`,
+      );
+      return false;
+    }
     return true;
   };
   const markUploadComplete = (
@@ -414,11 +425,11 @@ export default function useFileUpload(
         onChange={onFileChange}
         type="file"
         accept={
-          "allowAllExtensions" in options
-            ? "image/*,video/*,audio/*,text/plain,text/csv,application/rtf,application/msword,application/vnd.oasis.opendocument.text,application/pdf,application/vnd.ms-excel,application/vnd.openxmlformats-officedocument.spreadsheetml.sheet,application/vnd.oasis.opendocument.spreadsheet,application/pdf"
-            : "allowedExtensions" in options
-              ? options.allowedExtensions?.join(",")
-              : ""
+          "allowedExtensions" in options
+            ? options.allowedExtensions?.join(",")
+            : "allowAllExtensions" in options
+              ? "*"
+              : DEFAULT_ALLOWED_EXTENSIONS.join(",")
         }
         hidden
       />

From bb2ef69509cb109c03ca1c2fa12da972cd3f33c2 Mon Sep 17 00:00:00 2001
From: Rithvik Nishad <mail@rithviknishad.dev>
Date: Fri, 7 Jun 2024 12:46:30 +0530
Subject: [PATCH 10/11] Fix log update submit button disabled even after
 symptoms changed (#7992)

* Fix log update submit button disabled even after symptoms changed

* logic for checking symptoms changed
---
 src/Components/Patient/DailyRounds.tsx      | 65 ++++++++++++++-------
 src/Components/Symptoms/SymptomsBuilder.tsx | 24 ++++++--
 2 files changed, 61 insertions(+), 28 deletions(-)

diff --git a/src/Components/Patient/DailyRounds.tsx b/src/Components/Patient/DailyRounds.tsx
index 35e9828f9d1..dd68e1f7dd6 100644
--- a/src/Components/Patient/DailyRounds.tsx
+++ b/src/Components/Patient/DailyRounds.tsx
@@ -32,7 +32,6 @@ import routes from "../../Redux/api";
 import { Scribe } from "../Scribe/Scribe";
 import { DAILY_ROUND_FORM_SCRIBE_DATA } from "../Scribe/formDetails";
 import { DailyRoundsModel } from "./models";
-import { fetchEventTypeByName } from "../Facility/ConsultationDetails/Events/types";
 import InvestigationBuilder from "../Common/prescription-builder/InvestigationBuilder";
 import { FieldErrorText } from "../Form/FormFields/FormField";
 import { error } from "@pnotify/core";
@@ -61,6 +60,7 @@ const initForm: any = {
   systolic: null,
   investigations: [],
   investigations_dirty: false,
+  symptoms_dirty: false,
   diastolic: null,
   pulse: null,
   resp: null,
@@ -139,6 +139,7 @@ export const DailyRounds = (props: any) => {
     "review_interval",
     "bp",
     "pulse",
+    "temperature",
     "resp",
     "investigations",
     "ventilator_spo2",
@@ -149,7 +150,6 @@ export const DailyRounds = (props: any) => {
 
   const fetchRoundDetails = useCallback(async () => {
     setIsLoading(true);
-    fetchEventTypeByName("");
     let formData: any = initialData;
     if (id) {
       const { data } = await request(routes.getDailyReport, {
@@ -271,8 +271,7 @@ export const DailyRounds = (props: any) => {
     return !invalidForm;
   };
 
-  const handleSubmit = async (e: React.SyntheticEvent) => {
-    e.preventDefault();
+  const handleSubmit = async () => {
     const validForm = validateForm();
     if (validForm) {
       setIsLoading(true);
@@ -420,6 +419,32 @@ export const DailyRounds = (props: any) => {
     return <Loading />;
   }
 
+  const submitButtonDisabled = (() => {
+    if (buttonText !== "Save") {
+      return false;
+    }
+
+    if (["VENTILATOR", "DOCTORS_LOG"].includes(state.form.rounds_type)) {
+      return false;
+    }
+
+    if (state.form["symptoms_dirty"]) {
+      return false;
+    }
+
+    if (
+      formFields.every(
+        (field) =>
+          JSON.stringify(state.form[field]) ===
+          JSON.stringify(initialData[field]),
+      )
+    ) {
+      return true;
+    }
+
+    return false;
+  })();
+
   return (
     <Page
       title={headerText}
@@ -448,10 +473,7 @@ export const DailyRounds = (props: any) => {
           }}
         />
       </div>
-      <form
-        onSubmit={(e) => handleSubmit(e)}
-        className="w-full max-w-4xl rounded-lg bg-white px-8 py-5 shadow md:m-4 md:px-16 md:py-11"
-      >
+      <form className="w-full max-w-4xl rounded-lg bg-white px-8 py-5 shadow md:m-4 md:px-16 md:py-11">
         <DraftSection
           handleDraftSelect={(newState) => {
             dispatch({ type: "set_state", state: newState });
@@ -504,7 +526,14 @@ export const DailyRounds = (props: any) => {
         <div className="grid grid-cols-1 gap-x-6 md:grid-cols-2">
           <div className="pb-6 md:col-span-2">
             <FieldLabel>Symptoms</FieldLabel>
-            <EncounterSymptomsBuilder />
+            <EncounterSymptomsBuilder
+              onChange={() => {
+                handleFormFieldChange({
+                  name: "symptoms_dirty",
+                  value: true,
+                });
+              }}
+            />
           </div>
 
           <TextAreaFormField
@@ -722,19 +751,11 @@ export const DailyRounds = (props: any) => {
         <div className="mt-4 flex flex-col-reverse justify-end gap-2 md:flex-row">
           <Cancel onClick={() => goBack()} />
           <Submit
-            disabled={
-              buttonText === "Save" &&
-              formFields.every(
-                (field: string) =>
-                  JSON.stringify(state.form[field]) ===
-                  JSON.stringify(initialData[field]),
-              ) &&
-              (state.form.temperature == initialData.temperature ||
-                isNaN(state.form.temperature)) &&
-              state.form.rounds_type !== "VENTILATOR" &&
-              state.form.rounds_type !== "DOCTORS_LOG"
-            }
-            onClick={(e) => handleSubmit(e)}
+            disabled={submitButtonDisabled}
+            onClick={(e) => {
+              e.preventDefault();
+              handleSubmit();
+            }}
             label={buttonText}
           />
         </div>
diff --git a/src/Components/Symptoms/SymptomsBuilder.tsx b/src/Components/Symptoms/SymptomsBuilder.tsx
index de6901d8384..b4a11cf6999 100644
--- a/src/Components/Symptoms/SymptomsBuilder.tsx
+++ b/src/Components/Symptoms/SymptomsBuilder.tsx
@@ -68,7 +68,10 @@ export const CreateSymptomsBuilder = (props: {
   );
 };
 
-export const EncounterSymptomsBuilder = (props: { showAll?: boolean }) => {
+export const EncounterSymptomsBuilder = (props: {
+  showAll?: boolean;
+  onChange?: () => void;
+}) => {
   const consultationId = useSlug("consultation");
 
   const [isProcessing, setIsProcessing] = useState(false);
@@ -104,20 +107,26 @@ export const EncounterSymptomsBuilder = (props: { showAll?: boolean }) => {
         {items.map((symptom) => {
           const handleUpdate = async (event: FieldChangeEvent<unknown>) => {
             setIsProcessing(true);
-            await request(SymptomsApi.partialUpdate, {
+            const { res } = await request(SymptomsApi.partialUpdate, {
               pathParams: { consultationId, external_id: symptom.id },
               body: { [event.name]: event.value },
             });
-            await refetch();
+            if (res?.ok) {
+              props.onChange?.();
+              await refetch();
+            }
             setIsProcessing(false);
           };
 
           const handleMarkAsEnteredInError = async () => {
             setIsProcessing(true);
-            await request(SymptomsApi.markAsEnteredInError, {
+            const { res } = await request(SymptomsApi.markAsEnteredInError, {
               pathParams: { consultationId, external_id: symptom.id },
             });
-            await refetch();
+            if (res?.ok) {
+              props.onChange?.();
+              await refetch();
+            }
             setIsProcessing(false);
           };
 
@@ -147,7 +156,10 @@ export const EncounterSymptomsBuilder = (props: { showAll?: boolean }) => {
         <AddSymptom
           existing={data.results}
           consultationId={consultationId}
-          onAdd={() => refetch()}
+          onAdd={() => {
+            props.onChange?.();
+            refetch();
+          }}
         />
       </div>
     </div>

From d9e8747c88c544cbfa5289bfedc3839f9a384f82 Mon Sep 17 00:00:00 2001
From: Rithvik Nishad <mail@rithviknishad.dev>
Date: Fri, 7 Jun 2024 13:47:51 +0530
Subject: [PATCH 11/11] show uncategorised text for diagnosis without chapters
 (#7991)

* show uncategorised text for diagnosis without chapters

* fix uncategorised text shown if no principal diagnosis selected

* minor code improvement
---
 .../PrincipalDiagnosisSelect.tsx               | 18 ++++++++++++------
 src/Components/Diagnosis/types.ts              |  2 +-
 2 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/src/Components/Diagnosis/ConsultationDiagnosisBuilder/PrincipalDiagnosisSelect.tsx b/src/Components/Diagnosis/ConsultationDiagnosisBuilder/PrincipalDiagnosisSelect.tsx
index 8018c6480ed..657451e4538 100644
--- a/src/Components/Diagnosis/ConsultationDiagnosisBuilder/PrincipalDiagnosisSelect.tsx
+++ b/src/Components/Diagnosis/ConsultationDiagnosisBuilder/PrincipalDiagnosisSelect.tsx
@@ -48,12 +48,18 @@ const PrincipalDiagnosisSelect = <T extends Option>(props: Props<T>) => {
           }}
           errorClassName="hidden"
         />
-        {diagnosis && (
-          <span className="mt-3 flex w-full flex-wrap justify-center gap-x-1 px-2 text-center text-gray-900">
-            <p>This encounter will be categorised under:</p>
-            <p className="font-bold">{diagnosis.chapter}</p>
-          </span>
-        )}
+        {diagnosis &&
+          (diagnosis.chapter ? (
+            <span className="mt-3 flex w-full flex-wrap justify-center gap-x-1 px-2 text-center text-gray-900">
+              <p>This encounter will be categorised under:</p>
+              <p className="font-bold">{diagnosis.chapter}</p>
+            </span>
+          ) : (
+            <span className="mt-3 flex w-full flex-wrap justify-center gap-x-1 px-2 text-center italic text-gray-700">
+              This encounter will not be categorised under any chapter as the
+              diagnosis does not fall under a chapter.
+            </span>
+          ))}
       </div>
     </div>
   );
diff --git a/src/Components/Diagnosis/types.ts b/src/Components/Diagnosis/types.ts
index 006bb2feb6c..694e7a482d6 100644
--- a/src/Components/Diagnosis/types.ts
+++ b/src/Components/Diagnosis/types.ts
@@ -3,7 +3,7 @@ import { PerformedByModel } from "../HCX/misc";
 export type ICD11DiagnosisModel = {
   id: string;
   label: string;
-  chapter: string;
+  chapter?: string;
 };
 
 export const ActiveConditionVerificationStatuses = [