Enhance dataset mutation response structure and error handling in EditMetadata and Content components

Author: sanjaypinna

app/[locale]/dashboard/[entityType]/[entitySlug]/dataset/[id]/edit/components/EditMetadata.tsx

@@ -96,8 +96,13 @@ const metadataQueryDoc: any = graphql(`
 const updateMetadataMutationDoc: any = graphql(`
   mutation SaveMetadata($UpdateMetadataInput: UpdateMetadataInput!) {
     addUpdateDatasetMetadata(updateMetadataInput: $UpdateMetadataInput) {
-      __typename
-      ... on TypeDataset {
+      success
+      errors {
+        fieldErrors {
+          messages
+        }
+      }
+      data {
         id
         description
         title
@@ -215,13 +220,15 @@ export function EditMetadata({ id }: { id: string }) {
             `metadata_fields_list_${id}`,
           ],
         });
-        const updatedData = defaultValuesPrepFn(res.addUpdateDatasetMetadata);
+        const updatedData = defaultValuesPrepFn(
+          res.addUpdateDatasetMetadata.data
+        );
         setFormData(updatedData);
         setPreviousFormData(updatedData);
         // getDatasetMetadata.refetch();
       },
       onError: (err: any) => {
-        toast('Error:  ' + err.message.split(':')[0]);
+        toast('Error:  ' + err.message);
       },
     }
   );
@@ -425,6 +432,7 @@ export function EditMetadata({ id }: { id: string }) {
           <Input
             type="date"
             name={metadataFormItem.id}
+            max={new Date().toISOString().split('T')[0]}
             value={formData[metadataFormItem.id] || ''}
             label={metadataFormItem.label}
             disabled={
@@ -490,10 +498,11 @@ export function EditMetadata({ id }: { id: string }) {
                 <div className="w-full">
                   <TextField
                     key="description"
-                    multiline={3}
+                    multiline={4}
                     name="description"
                     label={'Description'}
                     value={formData.description}
+                    helpText="Character limit: 1000"
                     onChange={(e) => handleChange('description', e)}
                     onBlur={() => handleSave(formData)} // Save on blur
                   />

app/[locale]/dashboard/[entityType]/[entitySlug]/dataset/components/content.tsx

@@ -10,19 +10,19 @@ import { GraphQL } from '@/lib/api';
 import { Icons } from '@/components/icons';
 
 const createDatasetMutationDoc: any = graphql(`
-  mutation GenerateDatasetName {
+  mutation Generate_Dataset_Name {
     addDataset {
-      __typename
-      ... on TypeDataset {
+      success
+      errors {
+        fieldErrors {
+          messages
+        }
+      }
+      data {
         id
+        title
         created
       }
-      ... on OperationInfo {
-        messages {
-          kind
-          message
-        }
-      }
     }
   }
 `);
@@ -46,7 +46,7 @@ export const Content = ({
       {
         onSuccess: (data: any) => {
           router.push(
-            `/dashboard/${params.entityType}/${params.entitySlug}/dataset/${data?.addDataset?.id}/edit/metadata`
+            `/dashboard/${params.entityType}/${params.entitySlug}/dataset/${data?.addDataset?.data?.id}/edit/metadata`
           );
         },
         onError: (err: any) => {

app/[locale]/dashboard/[entityType]/[entitySlug]/dataset/page-layout.tsx

@@ -9,19 +9,19 @@ import { ActionBar } from './components/action-bar';
 import { Content } from './components/content';
 
 const createDatasetMutationDoc: any = graphql(`
-  mutation GenerateDatasetName {
+  mutation GenerateDatasetname {
     addDataset {
-      __typename
-      ... on TypeDataset {
+      success
+      errors {
+        fieldErrors {
+          messages
+        }
+      }
+      data {
         id
+        title
         created
       }
-      ... on OperationInfo {
-        messages {
-          kind
-          message
-        }
-      }
     }
   }
 `);

app/[locale]/dashboard/[entityType]/[entitySlug]/dataset/page.tsx

@@ -29,17 +29,17 @@ const allDatasetsQueryDoc: any = graphql(`
 const createDatasetMutationDoc: any = graphql(`
   mutation GenerateDatasetName {
     addDataset {
-      __typename
-      ... on TypeDataset {
+      success
+      errors {
+        fieldErrors {
+          messages
+        }
+      }
+      data {
         id
+        title
         created
       }
-      ... on OperationInfo {
-        messages {
-          kind
-          message
-        }
-      }
     }
   }
 `);
@@ -86,25 +86,19 @@ export default function DatasetPage({
   ];
 
   const AllDatasetsQuery: { data: any; isLoading: boolean; refetch: any } =
-    useQuery(
-      [`fetch_datasets_org_dashboard`],
-      () =>
-        GraphQL(
-          allDatasetsQueryDoc,
-          {
-            [params.entityType]: params.entitySlug,
+    useQuery([`fetch_datasets_org_dashboard`], () =>
+      GraphQL(
+        allDatasetsQueryDoc,
+        {
+          [params.entityType]: params.entitySlug,
+        },
+        {
+          filters: {
+            status: navigationTab === 'published' ? 'PUBLISHED' : 'DRAFT',
           },
-          {
-            filters: {
-              status: navigationTab === 'published' ? 'PUBLISHED' : 'DRAFT',
-            },
-            order: { modified: 'DESC' },
-          }
-        ),
-      {
-        refetchOnMount: true,
-        refetchOnReconnect: true,
-      }
+          order: { modified: 'DESC' },
+        }
+      )
     );
 
   useEffect(() => {
@@ -149,8 +143,9 @@ export default function DatasetPage({
         ),
       {
         onSuccess: (data: any) => {
+
           router.push(
-            `/dashboard/${params.entityType}/${params.entitySlug}/dataset/${data?.addDataset?.id}/edit/metadata`
+            `/dashboard/${params.entityType}/${params.entitySlug}/dataset/${data?.addDataset?.data?.id}/edit/metadata`
           );
         },
         onError: (err: any) => {