Feedback from code reviews

src/qqq/components/QDynamicForm/utils/DynamicFormUtils.ts

@@ -93,27 +93,6 @@ class DynamicFormUtils
       }
       return (null);
    }
-
-   public static getFormDataForUploadForm(fieldName: string, fieldLabel: string, isRequired: boolean = true)
-   {
-      const dynamicFormFields: any = {};
-      const formValidations: any = {};
-
-      dynamicFormFields[fieldName] = {
-         name: fieldName,
-         label: fieldLabel,
-         isRequired: isRequired,
-         type: "file",
-         // todo invalidMsg: "Zipcode is not valid (e.g. 70000).",
-      };
-
-      if (isRequired)
-      {
-         formValidations[fieldName] = Yup.string().required(`${fieldLabel} is required.`);
-      }
-
-      return {dynamicFormFields, formValidations};
-   }
 }
 
 export default DynamicFormUtils;

src/qqq/pages/entity-list/index.tsx

@@ -475,7 +475,7 @@ function EntityList({table}: Props): JSX.Element
 
    const bulkLoadClicked = () =>
    {
-      document.location.href = `/processes/${tableName}.bulkInsert${getRecordsQueryString()}`;
+      document.location.href = `/processes/${tableName}.bulkInsert`;
    };
 
    const bulkEditClicked = () =>
@@ -630,7 +630,15 @@ function EntityList({table}: Props): JSX.Element
          <MDBox my={3}>
             {alertContent ? (
                <MDBox mb={3}>
-                  <Alert severity="error">{alertContent}</Alert>
+                  <Alert
+                     severity="error"
+                     onClose={() =>
+                     {
+                        setAlertContent(null);
+                     }}
+                  >
+                     {alertContent}
+                  </Alert>
                </MDBox>
             ) : (
                ""