Refactor ManageQuestionnaireOrganizationsSheet and organizationApi types

- Simplified the mutation function in ManageQuestionnaireOrganizationsSheet for setting organizations by removing redundant parameters.
- Updated the handleSave function to correctly format the payload for the setOrganizations mutation.
- Enhanced the hasChanges logic to improve clarity and accuracy in detecting changes in selected organizations.
- Refactored organizationApi type definitions to use Type<> for response and request bodies, improving type safety and consistency across API calls.

src/components/Questionnaire/ManageQuestionnaireOrganizationsSheet.tsx

@@ -63,11 +63,9 @@ export default function ManageQuestionnaireOrganizationsSheet({
     });
 
   const { mutate: setOrganizations, isPending: isUpdating } = useMutation({
-    mutationFn: (organizations: string[]) =>
-      mutate(questionnaireApi.setOrganizations, {
-        pathParams: { id: questionnaireId },
-        body: { organizations: organizations },
-      })({ organizations: organizations }),
+    mutationFn: mutate(questionnaireApi.setOrganizations, {
+      pathParams: { id: questionnaireId },
+    }),
     onSuccess: () => {
       queryClient.invalidateQueries({
         queryKey: ["questionnaire", questionnaireId, "organizations"],
@@ -93,16 +91,18 @@ export default function ManageQuestionnaireOrganizationsSheet({
   };
 
   const handleSave = () => {
-    setOrganizations(selectedIds);
+    setOrganizations({ organizations: selectedIds });
   };
 
   const selectedOrganizations = organizations?.results.filter((org) =>
     selectedIds.includes(org.id),
   );
 
-  const hasChanges =
-    JSON.stringify(organizations?.results.map((org) => org.id).sort()) !==
-    JSON.stringify(selectedIds.sort());
+  const hasChanges = !organizations?.results
+    ? false
+    : new Set(organizations.results.map((org) => org.id)).size !==
+        new Set(selectedIds).size ||
+      !organizations.results.every((org) => selectedIds.includes(org.id));
 
   return (
     <Sheet open={open} onOpenChange={setOpen}>

src/types/organization/organizationApi.ts

@@ -8,39 +8,39 @@ export default {
   listMine: {
     path: "/api/v1/organization/mine/",
     method: HttpMethod.GET,
-    TRes: {} as PaginatedResponse<Organization>,
+    TRes: Type<PaginatedResponse<Organization>>(),
   },
   list: {
     path: "/api/v1/organization/",
     method: HttpMethod.GET,
-    TRes: {} as PaginatedResponse<Organization>,
+    TRes: Type<PaginatedResponse<Organization>>(),
   },
   get: {
     path: "/api/v1/organization/{id}/",
     method: HttpMethod.GET,
-    TRes: {} as Organization,
+    TRes: Type<Organization>(),
   },
   listUsers: {
     path: "/api/v1/organization/{id}/users/",
     method: HttpMethod.GET,
-    TRes: {} as PaginatedResponse<OrganizationUserRole>,
+    TRes: Type<PaginatedResponse<OrganizationUserRole>>(),
   },
   assignUser: {
     path: "/api/v1/organization/{id}/users/",
     method: HttpMethod.POST,
-    TRes: {} as OrganizationUserRole,
-    TBody: {} as { user: string; role: string },
+    TRes: Type<OrganizationUserRole>(),
+    TBody: Type<{ user: string; role: string }>(),
   },
   updateUserRole: {
     path: "/api/v1/organization/{id}/users/{userRoleId}/",
     method: HttpMethod.PUT,
-    TRes: {} as OrganizationUserRole,
-    TBody: {} as { user: string; role: string },
+    TRes: Type<OrganizationUserRole>(),
+    TBody: Type<{ user: string; role: string }>(),
   },
   removeUserRole: {
     path: "/api/v1/organization/{id}/users/{userRoleId}/",
     method: HttpMethod.DELETE,
-    TRes: {} as Record<string, never>,
+    TRes: Type<void>(),
   },
   listPatients: {
     path: "/api/v1/patient/",