Fix Modal Props actions to required false (#632)

nordcloud · Aug 2, 2023 · 8351a7c · 8351a7c
1 parent 21f5d7c
commit 8351a7c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@nordcloud/gnui",
   "description": "Nordcloud Design System - a collection of reusable React components used in Nordcloud's SaaS products",
-  "version": "8.13.2",
+  "version": "8.13.3",
   "license": "MIT",
   "repository": {
     "type": "git",

diff --git a/src/components/modal/Modal.tsx b/src/components/modal/Modal.tsx
@@ -53,10 +53,10 @@ export function Modal({
             >
               {children}
             </ModalContent>
-            {actions?.length > 0 && (
+            {actions && (
               <ModalActions>
                 {actions
-                  .sort((a, b) => a.order || 0 - b.order || 0)
+                  ?.sort((a, b) => a.order || 0 - b.order || 0)
                   .map((action, index) => (
                     <ModalAction
                       key={`modal-action-${index}`}

diff --git a/src/components/modal/types.ts b/src/components/modal/types.ts
@@ -22,7 +22,7 @@ type ModalAction = {
 export type ModalProps = {
   isOpen: boolean;
   onClose?: (event: React.KeyboardEvent | React.MouseEvent) => void;
-  actions: ModalAction[];
+  actions?: ModalAction[];
   contentLabel: string;
   children: React.ReactNode;
 };