fix(PN-9835): clear file name when remove a document on new notificat…

…ion attachments (#1183)
pagopa · Apr 15, 2024 · 32d7b6e · 32d7b6e
1 parent a78d784
commit 32d7b6e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/pn-pa-webapp/src/components/NewNotification/Attachments.tsx b/packages/pn-pa-webapp/src/components/NewNotification/Attachments.tsx
@@ -245,7 +245,10 @@ const Attachments: React.FC<Props> = ({
         key: '',
         versionToken: '',
       },
+      name: '',
     });
+
+    await formik.setFieldTouched(`${id}.name`, false, false);
   };
 
   const addDocumentHandler = async () => {

diff --git a/packages/pn-pa-webapp/src/components/NewNotification/__test__/Attachments.test.tsx b/packages/pn-pa-webapp/src/components/NewNotification/__test__/Attachments.test.tsx
@@ -188,9 +188,8 @@ describe('Attachments Component with payment enabled', async () => {
       expect(buttonSubmit).toBeDisabled();
     });
     await testInput(form!, `documents.0.name`, '');
-    const error = form!.querySelector(`[id="documents.0.name-helper-text"]`);
-    expect(error).toHaveTextContent('required-field');
     await testInput(form!, `documents.0.name`, ' text-with-spaces ');
+    const error = form!.querySelector(`[id="documents.0.name-helper-text"]`);
     expect(error).toHaveTextContent('no-spaces-at-edges');
   });