Merge pull request #920 from adarshlilha/bugfix/environment-save-empt…

…y-secret fix(#904): fix environment save when empty secret exists
usebruno · Nov 18, 2023 · a7e2f08 · a7e2f08
2 parents 30a2eb5 + 4358470
commit a7e2f08
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...ents/EnvironmentSettings/EnvironmentList/EnvironmentDetails/EnvironmentVariables/index.js b/...ents/EnvironmentSettings/EnvironmentList/EnvironmentDetails/EnvironmentVariables/index.js
@@ -32,7 +32,7 @@ const EnvironmentVariables = ({ environment, collection }) => {
         secret: Yup.boolean(),
         type: Yup.string(),
         uid: Yup.string(),
-        value: Yup.string().trim()
+        value: Yup.string().trim().nullable()
       })
     ),
     onSubmit: (values) => {
@@ -114,7 +114,7 @@ const EnvironmentVariables = ({ environment, collection }) => {
                   className="mousetrap"
                   id={`${index}.name`}
                   name={`${index}.name`}
-                  value={formik.values[index].name}
+                  value={variable.name}
                   onChange={formik.handleChange}
                 />
                 <ErrorMessage name={`${index}.name`} />