fix: allow empty values for config fields that default to an empty st…

…ring, closes #9
Longridge-High-School · Jun 27, 2023 · bccd5cc · bccd5cc
1 parent 9dd0677
commit bccd5cc
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/app/routes/admin.config.tsx b/app/routes/admin.config.tsx
@@ -49,11 +49,13 @@ export const action: ActionFunction = async ({request}) => {
   await asyncForEach(keys(ConfigurableValues), async key => {
     const value = formData.get(key) as string | undefined
 
-    invariant(value)
+    if (ConfigurableValues[key].value !== '') {
+      invariant(value)
+    }
 
     const currentValue = await getConfigValue(key)
 
-    if (value !== currentValue) {
+    if (value !== currentValue && value !== undefined) {
       await setConfigValue(key, value)
     }
   })