Merge pull request #56 from thjennifer/main

minor fix for dashboard navigation
looker-open-source · Oct 1, 2024 · d0dbed6 · d0dbed6
2 parents d7ca09e + 71d9412
commit d0dbed6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/RELEASE_NOTES.md b/RELEASE_NOTES.md
@@ -1,3 +1,7 @@
+## Release v6.0
+* Small fix to navigation template links.
+
+
 ## Release v5.4.1
 * Update README with message about using Balance Sheet and Income Statement with SAP Financial test harness data available with Cortext Framework.
 

diff --git a/views/navigation_template.view.lkml b/views/navigation_template.view.lkml
@@ -165,6 +165,7 @@ view: navigation_template {
     hidden: no
     description: "Add to Single Value Visualization. Defined HTML styling will be shown."
     sql: '' ;;
+    required_fields: [item_delimiter, value_delimiter]
     html:
      <!-- initial splits -->
       {% assign navItems = dash_bindings._value | split: item_delimiter._value %}
@@ -309,4 +310,4 @@ view: navigation_template {
   }
 
 
-}
+}