fix: service modal yup, breakout component, cleanup

services/frontend/src/components/Modal/Service/Create.tsx

@@ -1,5 +1,6 @@
 import { useState } from "react";
-import { useFormik } from "formik";
+import { Formik } from "formik";
+import * as yup from "yup";
 import { XIcon } from "@heroicons/react/outline";
 import General from "./General";
 import Environment from "./Environment";
@@ -16,22 +17,23 @@ interface IModalServiceProps {
 const ModalServiceCreate = (props: IModalServiceProps) => {
   const { onHide, onAddEndpoint } = props;
   const [openTab, setOpenTab] = useState("General");
-
+  const handleCreate = (values: any, formik: any) => {
-  const formik = useFormik({
+    onAddEndpoint(values);
-    initialValues: {
+    formik.resetForm();
-      canvasConfig: {
+  };
-        ...serviceConfigCanvasInitialValues()
+  const validationSchema = yup.object({
-      },
+    canvasConfig: yup.object({
-      serviceConfig: {
+      service_name: yup
-        container_name: ""
+        .string()
-      },
+        .max(256, "service name should be 256 characters or less")
-      key: "service",
+        .required("service name is required")
-      type: "SERVICE",
+    }),
-      inputs: ["op_source"],
+    serviceConfig: yup.object({
-      outputs: [],
+      container_name: yup
-      config: {}
+        .string()
-    },
+        .max(256, "container name should be 256 characters or less")
-    onSubmit: () => undefined
+        .required("container name is required")
+    })
   });
   const tabs = [
     {
@@ -84,7 +86,28 @@ const ModalServiceCreate = (props: IModalServiceProps) => {
               </button>
             </div>
 
-            <div>
+            <Formik
+              initialValues={{
+                canvasConfig: {
+                  ...serviceConfigCanvasInitialValues()
+                },
+                serviceConfig: {
+                  container_name: ""
+                },
+                key: "service",
+                type: "SERVICE",
+                inputs: ["op_source"],
+                outputs: [],
+                config: {}
+              }}
+              enableReinitialize={true}
+              onSubmit={(values, formik) => {
+                handleCreate(values, formik);
+              }}
+              validationSchema={validationSchema}
+            >
+              {(formik) => (
+                <>
                   <div className="hidden sm:block">
                     <div className="border-b border-gray-200 px-8">
                       <nav className="-mb-px flex space-x-8" aria-label="Tabs">
@@ -113,13 +136,10 @@ const ModalServiceCreate = (props: IModalServiceProps) => {
                   </div>
 
                   <div className="relative px-4 py-3 flex-auto">
-                <form onSubmit={formik.handleSubmit}>
+                    {openTab === "General" && <General />}
-                  {openTab === "General" && <General formik={formik} />}
+                    {openTab === "Environment" && <Environment />}
-                  {openTab === "Environment" && <Environment formik={formik} />}
+                    {openTab === "Volumes" && <Volumes />}
-                  {openTab === "Volumes" && <Volumes formik={formik} />}
+                    {openTab === "Labels" && <Labels />}
-                  {openTab === "Labels" && <Labels formik={formik} />}
-                </form>
-              </div>
                   </div>
 
                   <div className="flex items-center justify-end px-4 py-3 border-t border-solid border-blueGray-200 rounded-b">
@@ -134,6 +154,9 @@ const ModalServiceCreate = (props: IModalServiceProps) => {
                       Add
                     </button>
                   </div>
+                </>
+              )}
+            </Formik>
           </div>
         </div>
       </div>

services/frontend/src/components/Modal/Service/Edit.tsx

@@ -1,17 +1,12 @@
 import { useState, useEffect } from "react";
-import { useFormik } from "formik";
+import { Formik } from "formik";
+import * as yup from "yup";
 import { XIcon } from "@heroicons/react/outline";
 import General from "./General";
 import Environment from "./Environment";
 import Volumes from "./Volumes";
 import Labels from "./Labels";
-import { serviceConfigCanvasInitialValues } from "../../../utils";
+import { CallbackFunction, IServiceNodeItem } from "../../../types";
-import {
-  CallbackFunction,
-  ICanvasConfig,
-  IServiceNodeItem,
-  IService
-} from "../../../types";
 
 interface IModalServiceProps {
   node: IServiceNodeItem;
@@ -23,16 +18,26 @@ const ModalServiceEdit = (props: IModalServiceProps) => {
   const { node, onHide, onUpdateEndpoint } = props;
   const [openTab, setOpenTab] = useState("General");
   const [selectedNode, setSelectedNode] = useState<IServiceNodeItem>();
-  const formik = useFormik({
+
-    initialValues: {
+  const handleUpdate = (values: any) => {
-      canvasConfig: {
+    const updated = { ...selectedNode };
-        ...serviceConfigCanvasInitialValues()
+    updated.canvasConfig = values.canvasConfig;
-      },
+    updated.serviceConfig = values.serviceConfig;
-      serviceConfig: {
+    onUpdateEndpoint(updated);
-        container_name: ""
+  };
-      }
+  const validationSchema = yup.object({
-    },
+    canvasConfig: yup.object({
-    onSubmit: () => undefined
+      service_name: yup
+        .string()
+        .max(256, "service name should be 256 characters or less")
+        .required("service name is required")
+    }),
+    serviceConfig: yup.object({
+      container_name: yup
+        .string()
+        .max(256, "container name should be 256 characters or less")
+        .required("container name is required")
+    })
   });
   const tabs = [
     {
@@ -70,25 +75,6 @@ const ModalServiceEdit = (props: IModalServiceProps) => {
     }
   }, [node]);
 
-  useEffect(() => {
-    formik.resetForm();
-
-    if (selectedNode) {
-      formik.initialValues.canvasConfig = {
-        ...selectedNode.canvasConfig
-      } as ICanvasConfig;
-      formik.initialValues.serviceConfig = {
-        ...selectedNode.serviceConfig
-      } as IService;
-    }
-  }, [selectedNode]);
-
-  useEffect(() => {
-    return () => {
-      formik.resetForm();
-    };
-  }, []);
-
   return (
     <div className="fixed z-50 inset-0 overflow-y-auto">
       <div className="justify-center items-center flex overflow-x-hidden overflow-y-auto fixed inset-0 outline-none focus:outline-none">
@@ -110,10 +96,30 @@ const ModalServiceEdit = (props: IModalServiceProps) => {
               </button>
             </div>
 
-            <div>
+            {selectedNode && (
+              <Formik
+                initialValues={{
+                  canvasConfig: {
+                    ...selectedNode.canvasConfig
+                  },
+                  serviceConfig: {
+                    ...selectedNode.serviceConfig
+                  }
+                }}
+                enableReinitialize={true}
+                onSubmit={(values) => {
+                  handleUpdate(values);
+                }}
+                validationSchema={validationSchema}
+              >
+                {(formik) => (
+                  <>
                     <div className="hidden sm:block">
                       <div className="border-b border-gray-200 px-8">
-                  <nav className="-mb-px flex space-x-8" aria-label="Tabs">
+                        <nav
+                          className="-mb-px flex space-x-8"
+                          aria-label="Tabs"
+                        >
                           {tabs.map((tab) => (
                             <a
                               key={tab.name}
@@ -139,13 +145,10 @@ const ModalServiceEdit = (props: IModalServiceProps) => {
                     </div>
 
                     <div className="relative px-4 py-3 flex-auto">
-                <form onSubmit={formik.handleSubmit}>
+                      {openTab === "General" && <General />}
-                  {openTab === "General" && <General formik={formik} />}
+                      {openTab === "Environment" && <Environment />}
-                  {openTab === "Environment" && <Environment formik={formik} />}
+                      {openTab === "Volumes" && <Volumes />}
-                  {openTab === "Volumes" && <Volumes formik={formik} />}
+                      {openTab === "Labels" && <Labels />}
-                  {openTab === "Labels" && <Labels formik={formik} />}
-                </form>
-              </div>
                     </div>
 
                     <div className="flex items-center justify-end px-4 py-3 border-t border-solid border-blueGray-200 rounded-b">
@@ -153,15 +156,16 @@ const ModalServiceEdit = (props: IModalServiceProps) => {
                         className="btn-util"
                         type="button"
                         onClick={() => {
-                  const updated = { ...selectedNode };
+                          formik.submitForm();
-                  updated.canvasConfig = formik.values.canvasConfig;
-                  updated.serviceConfig = formik.values.serviceConfig;
-                  onUpdateEndpoint(updated);
                         }}
                       >
                         Update
                       </button>
                     </div>
+                  </>
+                )}
+              </Formik>
+            )}
           </div>
         </div>
       </div>

services/frontend/src/components/Modal/Service/Environment.tsx

@@ -1,6 +1,5 @@
-const Environment = (props: any) => {
+const Environment = () => {
-  const { formik } = props;
-
   return <></>;
 };
+
 export default Environment;

services/frontend/src/components/Modal/Service/General.tsx

@@ -1,56 +1,12 @@
-const General = (props: any) => {
+import TextField from "../../global/FormElements/InputField";
-  const { formik } = props;
 
+const General = () => {
   return (
     <>
-      <div className="relative pb-3 flex-auto">
+      <TextField label="Service name" name="canvasConfig.service_name" />
-        <div className="grid grid-cols-6 gap-4">
+      <TextField label="Container name" name="serviceConfig.container_name" />
-          <div className="col-span-3">
-            <label
-              htmlFor="service_name"
-              className="block text-xs font-medium text-gray-700"
-            >
-              Service name
-            </label>
-            <div className="mt-1">
-              <input
-                id="service_name"
-                name="canvasConfig.service_name"
-                type="text"
-                autoComplete="none"
-                className="input-util"
-                onChange={formik.handleChange}
-                value={formik.values.canvasConfig.service_name}
-              />
-            </div>
-          </div>
-        </div>
-      </div>
-
-      <div className="relative pb-3 flex-auto">
-        <div className="grid grid-cols-6 gap-4">
-          <div className="col-span-3">
-            <label
-              htmlFor="container_name"
-              className="block text-xs font-medium text-gray-700"
-            >
-              Container name
-            </label>
-            <div className="mt-1">
-              <input
-                id="container_name"
-                name="serviceConfig.container_name"
-                type="text"
-                autoComplete="none"
-                className="input-util"
-                onChange={formik.handleChange}
-                value={formik.values.serviceConfig.container_name}
-              />
-            </div>
-          </div>
-        </div>
-      </div>
     </>
   );
 };
+
 export default General;

services/frontend/src/components/Modal/Service/Labels.tsx

@@ -1,6 +1,5 @@
-const Labels = (props: any) => {
+const Labels = () => {
-  const { formik } = props;
-
   return <></>;
 };
+
 export default Labels;

services/frontend/src/components/Modal/Service/Volumes.tsx

@@ -1,6 +1,5 @@
-const Volumes = (props: any) => {
+const Volumes = () => {
-  const { formik } = props;
-
   return <></>;
 };
+
 export default Volumes;

services/frontend/src/components/Modal/Volume/Create.tsx

@@ -15,7 +15,7 @@ interface IModalVolumeCreate {
 const ModalVolumeCreate = (props: IModalVolumeCreate) => {
   const { onHide, onAddEndpoint } = props;
   const [openTab, setOpenTab] = useState("General");
-  const handleSave = (values: any, formik: any) => {
+  const handleCreate = (values: any, formik: any) => {
     onAddEndpoint(values);
     formik.resetForm();
   };
@@ -79,7 +79,7 @@ const ModalVolumeCreate = (props: IModalVolumeCreate) => {
               }}
               enableReinitialize={true}
               onSubmit={(values, formik) => {
-                handleSave(values, formik);
+                handleCreate(values, formik);
               }}
               validationSchema={validationSchema}
             >