Merge pull request #300 from performant-software/feature/cdc308_media_upload_error

dleadbetter · web-flow · commit a28829882c9f · 2024-10-03T12:16:50.000-04:00
CDC #308 - Media Upload Error
diff --git a/packages/controlled-vocabulary/package.json b/packages/controlled-vocabulary/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@performant-software/controlled-vocabulary",
-  "version": "2.2.14",
+  "version": "2.2.15",
   "description": "A package of components to allow user to configure dropdown elements. Use with the \"controlled_vocabulary\" gem.",
   "license": "MIT",
   "main": "./dist/index.cjs.js",
@@ -23,8 +23,8 @@
     "underscore": "^1.13.2"
   },
   "peerDependencies": {
-    "@performant-software/semantic-components": "^2.2.14",
-    "@performant-software/shared-components": "^2.2.14",
+    "@performant-software/semantic-components": "^2.2.15",
+    "@performant-software/shared-components": "^2.2.15",
     "react": ">= 16.13.1 < 19.0.0",
     "react-dom": ">= 16.13.1 < 19.0.0"
   },
diff --git a/packages/core-data/package.json b/packages/core-data/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@performant-software/core-data",
-  "version": "2.2.14",
+  "version": "2.2.15",
   "description": "A package of components used with the Core Data platform.",
   "license": "MIT",
   "main": "./dist/index.cjs.js",
@@ -40,8 +40,8 @@
     "underscore": "^1.13.2"
   },
   "peerDependencies": {
-    "@performant-software/geospatial": "^2.2.14",
-    "@performant-software/shared-components": "^2.2.14",
+    "@performant-software/geospatial": "^2.2.15",
+    "@performant-software/shared-components": "^2.2.15",
     "@peripleo/maplibre": "^0.5.2",
     "@peripleo/peripleo": "^0.5.2",
     "react": ">= 16.13.1 < 19.0.0",
diff --git a/packages/geospatial/package.json b/packages/geospatial/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@performant-software/geospatial",
-  "version": "2.2.14",
+  "version": "2.2.15",
   "description": "A package of components for all things map-related.",
   "license": "MIT",
   "main": "./dist/index.cjs.js",
diff --git a/packages/semantic-ui/package.json b/packages/semantic-ui/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@performant-software/semantic-components",
-  "version": "2.2.14",
+  "version": "2.2.15",
   "description": "A package of shared components based on the Semantic UI Framework.",
   "license": "MIT",
   "main": "./dist/index.cjs.js",
@@ -35,7 +35,7 @@
     "zotero-translation-client": "^5.0.1"
   },
   "peerDependencies": {
-    "@performant-software/shared-components": "^2.2.14",
+    "@performant-software/shared-components": "^2.2.15",
     "@samvera/clover-iiif": "^2.3.2",
     "react": ">= 16.13.1 < 19.0.0",
     "react-dnd": "^11.1.3",
diff --git a/packages/semantic-ui/src/components/FileUploadModal.js b/packages/semantic-ui/src/components/FileUploadModal.js
@@ -448,7 +448,7 @@ const FileUploadModal: ComponentType<any> = (props: Props) => {
                 </Message.List>
               </Message>
             )}
-            { props.errors && (
+            { !_.isEmpty(props.errors) && (
               <Message
                 error
               >
diff --git a/packages/shared/package.json b/packages/shared/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@performant-software/shared-components",
-  "version": "2.2.14",
+  "version": "2.2.15",
   "description": "A package of shared, framework agnostic, components.",
   "license": "MIT",
   "main": "./dist/index.cjs.js",
diff --git a/packages/storybook/src/semantic-ui/FileUploadModal.stories.js b/packages/storybook/src/semantic-ui/FileUploadModal.stories.js
@@ -51,7 +51,7 @@ export const Default = withWrapper((props) => (
   />
 ));
 
-export const Errors = withWrapper((props) => (
+export const InputErrors = withWrapper((props) => (
   <FileUploadModal
     closeOnComplete={false}
     itemComponent={({
@@ -98,6 +98,28 @@ export const Errors = withWrapper((props) => (
   />
 ));
 
+export const UploadErrors = withWrapper((props) => (
+  <FileUploadModal
+    errors={['File too large.']}
+    itemComponent={({ item }) => (
+      <Form.Input
+        label='Name'
+        value={item.name || ''}
+      />
+    )}
+    onAddFile={(file) => ({
+      name: file.name,
+      content: file,
+      type: file.type
+    })}
+    onClose={props.onClose}
+    onSave={() => {
+      action('save')();
+      return Promise.resolve();
+    }}
+  />
+));
+
 export const Images = withWrapper((props) => (
   <FileUploadModal
     itemComponent={({ item, onTextInputChange }) => (
diff --git a/packages/user-defined-fields/package.json b/packages/user-defined-fields/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@performant-software/user-defined-fields",
-  "version": "2.2.14",
+  "version": "2.2.15",
   "description": "A package of components used for allowing end users to define fields on models. Use with the \"user_defined_fields\" gem.",
   "license": "MIT",
   "main": "./dist/index.cjs.js",
@@ -23,8 +23,8 @@
     "underscore": "^1.13.2"
   },
   "peerDependencies": {
-    "@performant-software/semantic-components": "^2.2.14",
-    "@performant-software/shared-components": "^2.2.14",
+    "@performant-software/semantic-components": "^2.2.15",
+    "@performant-software/shared-components": "^2.2.15",
     "react": ">= 16.13.1 < 19.0.0",
     "react-dom": ">= 16.13.1 < 19.0.0"
   },
diff --git a/packages/visualize/package.json b/packages/visualize/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@performant-software/visualize",
-  "version": "2.2.14",
+  "version": "2.2.15",
   "description": "A package of components used for data visualization",
   "license": "MIT",
   "main": "./dist/index.cjs.js",
diff --git a/react-components.json b/react-components.json
@@ -8,5 +8,5 @@
     "packages/user-defined-fields",
     "packages/visualize"
   ],
-  "version": "2.2.14"
+  "version": "2.2.15"
 }

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@performant-software/geospatial",`
`3`		`- "version": "2.2.14",`
	`3`	`+ "version": "2.2.15",`
`4`	`4`	`"description": "A package of components for all things map-related.",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"main": "./dist/index.cjs.js",`
Original file line number	Diff line number	Diff line change
`@@ -448,7 +448,7 @@ const FileUploadModal: ComponentType<any> = (props: Props) => {`
`448`	`448`	`</Message.List>`
`449`	`449`	`</Message>`
`450`	`450`	`)}`
`451`		`- { props.errors && (`
	`451`	`+ { !_.isEmpty(props.errors) && (`
`452`	`452`	`<Message`
`453`	`453`	`error`
`454`	`454`	`>`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@performant-software/shared-components",`
`3`		`- "version": "2.2.14",`
	`3`	`+ "version": "2.2.15",`
`4`	`4`	`"description": "A package of shared, framework agnostic, components.",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"main": "./dist/index.cjs.js",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@performant-software/visualize",`
`3`		`- "version": "2.2.14",`
	`3`	`+ "version": "2.2.15",`
`4`	`4`	`"description": "A package of components used for data visualization",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"main": "./dist/index.cjs.js",`
Original file line number	Diff line number	Diff line change
`@@ -8,5 +8,5 @@`
`8`	`8`	`"packages/user-defined-fields",`
`9`	`9`	`"packages/visualize"`
`10`	`10`	`],`
`11`		`- "version": "2.2.14"`
	`11`	`+ "version": "2.2.15"`
`12`	`12`	`}`