Merge pull request #626 from bandada-infra/fix/dashboard-mult-credent…

…ials-expression fix(dashboard): missing credentials and expressions under certain conditions
bandada-infra · Dec 10, 2024 · eb0bfa1 · eb0bfa1
2 parents 404ad6a + 852595e
commit eb0bfa1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/apps/dashboard/src/components/new-group-stepper/access-mode-step.tsx b/apps/dashboard/src/components/new-group-stepper/access-mode-step.tsx
@@ -341,6 +341,14 @@ export default function AccessModeStep({
                                                 i.toString()
                                             ])
 
+                                            if (_credentials?.id) {
+                                                delete _credentials.id
+                                            }
+
+                                            if (_credentials?.criteria) {
+                                                delete _credentials.criteria
+                                            }
+
                                             const tempCredential = {
                                                 ..._credentials
                                             }

diff --git a/apps/dashboard/src/pages/new-group.tsx b/apps/dashboard/src/pages/new-group.tsx
@@ -28,7 +28,7 @@ export default function NewGroupPage(): JSX.Element {
             setCurrentStep((v) => v + 1)
         }
 
-        if (group.type === "on-chain" && !next) {
+        if (group?.type === "on-chain" && !next) {
             setGroup({ type: "on-chain", fingerprintDuration: 3600 })
         }
     }, [])