From 3dde03ec9f012a90d1fc5ea19f9f7e8ea38df406 Mon Sep 17 00:00:00 2001 From: ktx-vaidehi <134508096+ktx-vaidehi@users.noreply.github.com> Date: Tue, 23 Sep 2025 18:04:47 +0530 Subject: [PATCH] fix: update quick_mode parameter handling in logs URL based on configuration for drilldown and go to logs in dashboard (#8586) ### **PR Type** Bug fix ___ ### **Description** - Respect `quick_mode_enabled` in logs URL - Apply to drilldown and go-to-logs flows - Minor watch formatting cleanup ___ ### Diagram Walkthrough ```mermaid flowchart LR Config["zoConfig.quick_mode_enabled"] Builder1["PanelContainer: build logs URL"] Builder2["PanelSchemaRenderer: build logs URL"] URLParam["Set quick_mode=true/false"] Format["Refactor watch() formatting"] Config -- controls --> Builder1 Config -- controls --> Builder2 Builder1 -- sets --> URLParam Builder2 -- sets --> URLParam PanelSchemaRenderer --> Format ```

File Walkthrough

Relevant files
Bug fix
PanelContainer.vue
Conditional quick_mode in logs URL construction                   

web/src/components/dashboards/PanelContainer.vue
  • Set quick_mode based on zoConfig.quick_mode_enabled.
  • Remove unconditional quick_mode=false.
+5/-2     
PanelSchemaRenderer.vue
Apply config-driven quick_mode and tidy watchers                 

web/src/components/dashboards/PanelSchemaRenderer.vue
  • Set quick_mode based on zoConfig.quick_mode_enabled.
  • Keep histogram hidden; preserve other params.
  • Reformat watch() for readability.
+12/-5   
___ --- .../components/dashboards/PanelContainer.vue | 7 +++++-- .../dashboards/PanelSchemaRenderer.vue | 17 ++++++++++++----- 2 files changed, 17 insertions(+), 7 deletions(-) diff --git a/web/src/components/dashboards/PanelContainer.vue b/web/src/components/dashboards/PanelContainer.vue index e01912fc54..61cce23e5e 100644 --- a/web/src/components/dashboards/PanelContainer.vue +++ b/web/src/components/dashboards/PanelContainer.vue @@ -611,9 +611,12 @@ export default defineComponent({ "org_identifier", store.state.selectedOrganization.identifier, ); - logsUrl.searchParams.set("quick_mode", "false"); + if (store.state.zoConfig.quick_mode_enabled) { + logsUrl.searchParams.set("quick_mode", "true"); + } else { + logsUrl.searchParams.set("quick_mode", "false"); + } logsUrl.searchParams.set("show_histogram", "false"); - return logsUrl; }; diff --git a/web/src/components/dashboards/PanelSchemaRenderer.vue b/web/src/components/dashboards/PanelSchemaRenderer.vue index 0dc991742f..cd08c69fb4 100644 --- a/web/src/components/dashboards/PanelSchemaRenderer.vue +++ b/web/src/components/dashboards/PanelSchemaRenderer.vue @@ -747,10 +747,13 @@ export default defineComponent({ { deep: true }, ); - watch(panelData, () => { - emit("series-data-update", panelData.value); - }, - { deep: true }); + watch( + panelData, + () => { + emit("series-data-update", panelData.value); + }, + { deep: true }, + ); // when we get the new limitNumberOfSeriesWarningMessage from the convertPanelData, emit the limitNumberOfSeriesWarningMessage watch( @@ -1280,7 +1283,11 @@ export default defineComponent({ "org_identifier", store.state.selectedOrganization.identifier, ); - logsUrl.searchParams.set("quick_mode", "false"); + if (store.state.zoConfig.quick_mode_enabled) { + logsUrl.searchParams.set("quick_mode", "true"); + } else { + logsUrl.searchParams.set("quick_mode", "false"); + } logsUrl.searchParams.set("show_histogram", "false"); return logsUrl;