Fixes onSuccess mutation download batch

wri · Jan 16, 2025 · 7f5cd7a · 7f5cd7a
1 parent 98e35b8
commit 7f5cd7a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/ckan-backend-dev/src/ckanext-wri/ckanext/wri/logic/action/create.py b/ckan-backend-dev/src/ckanext-wri/ckanext/wri/logic/action/create.py
@@ -852,9 +852,7 @@ def download_event_create(context: Context, data_dict: DataDict):
 
     for item in [
         ["package_id", package_id],
-        ["resources", resources],
         ["email", email],
-        ["affiliation", affiliation],
     ]:
         if not item[1]:
             raise tk.ValidationError("Missing required field " + item[0])

diff --git a/deployment/frontend/src/components/datasets/sections/DataFiles.tsx b/deployment/frontend/src/components/datasets/sections/DataFiles.tsx
@@ -316,8 +316,8 @@ export function DataFiles({
         onError: (err) => {
             toast('Failed to send your information', {
                 type: 'error',
-            }),
-                setOpen(false)
+            })
+            setOpen(false)
         },
     })
 
@@ -333,15 +333,16 @@ export function DataFiles({
             },
             {
                 onSuccess: () => {
-                    toast("You'll receive an email when the file is ready", {
-                        type: 'success',
-                    })
                     const _data = {
                         ...data,
                         resources: keys,
                         package_id: dataset.id ?? '',
                     }
+                    console.log('Creating download event with data:', _data)
                     createDownloadEvent.mutate(_data)
+                    toast("You'll receive an email when the file is ready", {
+                        type: 'success',
+                    })
                     setOpen(false)
                 },
                 onError: (err) => {