fix(boards): issues with old-import board name containing spaces (#4052)

apps/nextjs/src/app/[locale]/boards/[name]/settings/page.tsx

@@ -61,6 +61,10 @@ const getBoardAndPermissionsAsync = async (params: Awaited<Props["params"]>) =>
       notFound();
     }
 
+    if (error instanceof TRPCError && error.code === "BAD_REQUEST") {
+      notFound();
+    }
+
     throw error;
   }
 };

apps/nextjs/src/app/[locale]/boards/_layout-creator.tsx

@@ -45,6 +45,10 @@ export const createBoardLayout = <TParams extends Params>({
         notFound();
       }
 
+      if (error instanceof TRPCError && error.code === "BAD_REQUEST") {
+        notFound();
+      }
+
       throw error;
     });
     const colorScheme = await getCurrentColorSchemeAsync();

packages/modals-collection/src/boards/import-board-modal.tsx

@@ -59,7 +59,7 @@ export const ImportBoardModal = createModal(({ actions }) => {
           }
 
           setFileValid(true);
-          form.setFieldValue("configuration.name", result.data.configProperties.name);
+          form.setFieldValue("configuration.name", result.data.configProperties.name.replaceAll(" ", "-"));
         })();
       },
     },

packages/old-import/src/analyse/analyse-oldmarr-import.ts

@@ -29,7 +29,7 @@ export const analyseOldmarrImportAsync = async (file: File) => {
     }
 
     return {
-      name: entry.name.replace(".json", ""),
+      name: entry.name.replaceAll(" ", "-").replace(".json", ""),
       config: result.data ?? null,
       isError: !result.success,
     };

packages/validation/src/board.ts

@@ -22,7 +22,7 @@ export const boardNameSchema = z
   .string()
   .min(1)
   .max(255)
-  .regex(/^[A-Za-z0-9-\\_]*$/);
+  .regex(/^[A-Za-z0-9-_]*$/);
 export const boardColumnCountSchema = z.number().min(1).max(24);
 
 export const boardByNameSchema = z.object({