backport of KV directory fixes (#24144)

Co-authored-by: Chelsea Shaw <[email protected]>
hashicorp · Nov 15, 2023 · 30e5521 · 30e5521
1 parent 23bed9e
commit 30e5521
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/ui/app/components/dashboard/quick-actions-card.js b/ui/app/components/dashboard/quick-actions-card.js
@@ -140,15 +140,17 @@ export default class DashboardQuickActionsCard extends Component {
   @action
   navigateToPage() {
     let route = this.searchSelectParams.route;
-    let param = this.paramValue.id;
+    // If search-select falls back to stringInput, paramVlue is a string not object
+    let param = this.paramValue.id || this.paramValue;
 
     // kv has a special use case where if the paramValue ends in a '/' you should
     // link to different route
     if (this.selectedEngine.type === 'kv') {
-      route = pathIsDirectory(this.paramValue?.path)
+      const path = this.paramValue.path || this.paramValue;
+      route = pathIsDirectory(path)
         ? 'vault.cluster.secrets.backend.kv.list-directory'
         : 'vault.cluster.secrets.backend.kv.secret.details';
-      param = this.paramValue?.path;
+      param = path;
     }
 
     this.router.transitionTo(route, this.selectedEngine.id, param);

diff --git a/ui/lib/kv/addon/components/page/list.hbs b/ui/lib/kv/addon/components/page/list.hbs
@@ -54,7 +54,7 @@
         <div class="level is-mobile">
           <div class="level-left">
             <div>
-              <Icon @name="file" class="has-text-grey-light" />
+              <Icon @name={{if metadata.pathIsDirectory "folder" "file"}} class="has-text-grey-light" />
               <span class="has-text-weight-semibold is-underline">
                 {{metadata.path}}
               </span>