fix(ui): single namespace properly filtered

kestra-io · Jul 19, 2024 · c0f8870 · c0f8870
1 parent b1a7353
commit c0f8870
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 19 deletions.
diff --git a/ui/src/components/namespace/Namespace.vue b/ui/src/components/namespace/Namespace.vue
@@ -60,11 +60,11 @@
             ...mapState("auth", ["user"]),
             ...mapState("namespace", ["dependencies"]),
             canCreateSecret() {
-                return this.$route.params.namespace && this.user &&
-                    this.user.isAllowed(permission.SECRET, action.CREATE, this.$route.params.namespace);
+                return this.$route.params.id && this.user &&
+                    this.user.isAllowed(permission.SECRET, action.CREATE, this.$route.params.id);
             },
             canCreateKv() {
-                return this.$route.params.namespace;
+                return this.$route.params.id;
             },
             routeInfo() {
                 return {
@@ -217,12 +217,8 @@
         },
         methods: {
             loadItem() {
-                if (this.$route.params.namespace) {
-                    this.$store
-                        .dispatch(
-                            "namespace/load",
-                            this.$route.params
-                        )
+                if (this.$route.params.id) {
+                    this.$store.dispatch("namespace/load",this.$route.params.id)
                 }
             },
         }

diff --git a/ui/src/components/namespace/NamespaceDependenciesWrapper.vue b/ui/src/components/namespace/NamespaceDependenciesWrapper.vue
@@ -1,5 +1,5 @@
 <template>
-    <NamespaceDependencies :id="route.params.namespace" />
+    <NamespaceDependencies :id="route.params.id" />
 </template>
 
 <script setup>

diff --git a/ui/src/components/namespace/NamespaceKV.vue b/ui/src/components/namespace/NamespaceKV.vue
@@ -139,10 +139,10 @@
                 return this.kv.key ? this.$t("kv.update", {key: this.kv.key}) : this.$t("kv.add");
             },
             canUpdateKv() {
-                return this.$route.params.namespace                
+                return this.$route.params.id                
             },
             canDeleteKv() {
-                return this.$route.params.namespace
+                return this.$route.params.id
             },
             addKvDrawerVisible: {
                 get() {
@@ -227,7 +227,7 @@
             },
             async updateKvModal(key) {
                 this.kv.key = key;
-                const {type, value} = await this.$store.dispatch("namespace/kv", {namespace: this.$route.params.namespace, key});
+                const {type, value} = await this.$store.dispatch("namespace/kv", {namespace: this.$route.params.id, key});
                 this.kv.type = type;
                 if (type === "JSON") {
                     this.kv.value = JSON.stringify(value);
@@ -242,7 +242,7 @@
             removeKv(key) {
                 this.$toast().confirm(this.$t("delete confirm", {name: key}), () => {
                     return this.$store
-                        .dispatch("namespace/deleteKv", {namespace: this.$route.params.namespace, key: key})
+                        .dispatch("namespace/deleteKv", {namespace: this.$route.params.id, key: key})
                         .then(() => {
                             this.$toast().deleted(key);
                         });
@@ -269,7 +269,7 @@
                     return this.$store
                         .dispatch(
                             "namespace/createKv",
-                            {namespace: this.$route.params.namespace, ...this.kv, value}
+                            {namespace: this.$route.params.id, ...this.kv, value}
                         )
                         .then(() => {
                             this.$toast().saved(this.kv.key);

diff --git a/ui/src/components/namespace/Overview.vue b/ui/src/components/namespace/Overview.vue
@@ -1,9 +1,9 @@
 <template>
     <div>
         <home
-            :namespace="$route.params.namespace || $route.query.id"
+            :namespace="$route.params.id || $route.query.id"
             :restore-url="false"
-            :description="$route.params.namespace ? namespace?.description : undefined"
+            :description="$route.params.id ? namespace?.description : undefined"
             embed
         />
     </div>

diff --git a/ui/src/stores/namespaces.js b/ui/src/stores/namespaces.js
@@ -27,8 +27,8 @@ export default {
                     return response.data.results;
                 })
         },
-        load({commit}, options) {
-            return this.$http.get(`${apiUrl(this)}/namespaces/${options.id}`, VALIDATE)
+        load({commit}, id) {
+            return this.$http.get(`${apiUrl(this)}/namespaces/${id}`, VALIDATE)
                 .then(response => {
                     if(response.status === 200) commit("setNamespace", response.data)
                     return response.data;