CR Update: Improve ohifmodal proptypes

platform/ui/src/components/ohifModal/OHIFModal.js

@@ -54,7 +54,12 @@ OHIFModal.propTypes = {
   onHide: PropTypes.func,
   footer: PropTypes.node,
   header: PropTypes.node,
-  children: PropTypes.node,
+  children: PropTypes.oneOfType([
+    PropTypes.arrayOf(PropTypes.node),
+    PropTypes.node,
+    PropTypes.object,
+    PropTypes.func,
+  ]).isRequired,
 };
 
 export default OHIFModal;

platform/ui/src/utils/ModalProvider.js

@@ -40,10 +40,9 @@ const ModalProvider = ({ children, modal: Modal, service }) => {
    *
    * @returns void
    */
-  const show = useCallback((component, props = {}) => {
+  const show = useCallback((component, props = {}) =>
-    setOptions(Object.assign({}, options, props, { component }));
+    setOptions(Object.assign({}, options, props, { component }))
-    console.log(component, props);
+  );
-  });
 
   /**
    * Hide the modal and set its properties to default.
@@ -80,8 +79,16 @@ const ModalProvider = ({ children, modal: Modal, service }) => {
 };
 
 ModalProvider.propTypes = {
-  children: PropTypes.node.isRequired,
+  children: PropTypes.oneOfType([
-  modal: PropTypes.node.isRequired,
+    PropTypes.arrayOf(PropTypes.node),
+    PropTypes.node,
+    PropTypes.func,
+  ]).isRequired,
+  modal: PropTypes.oneOfType([
+    PropTypes.arrayOf(PropTypes.node),
+    PropTypes.node,
+    PropTypes.func,
+  ]).isRequired,
   service: PropTypes.shape({
     setServiceImplementation: PropTypes.func,
   }).isRequired,