This is an automated email from the ASF dual-hosted git repository.

villebro pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new efd7007  fix(native-filters): show overridden chart name on scoping 
tree (#15038)
efd7007 is described below

commit efd70077014cbed62e493372d33a2af5237eaadf
Author: Ville Brofeldt <[email protected]>
AuthorDate: Tue Jun 8 14:55:11 2021 +0300

    fix(native-filters): show overridden chart name on scoping tree (#15038)
---
 .../FiltersConfigModal/FiltersConfigForm/FilterScope/utils.ts       | 6 +++++-
 superset-frontend/src/dashboard/types.ts                            | 1 +
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git 
a/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/FilterScope/utils.ts
 
b/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/FilterScope/utils.ts
index 8c74aa7..8cb48c1 100644
--- 
a/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/FilterScope/utils.ts
+++ 
b/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/FilterScope/utils.ts
@@ -45,7 +45,11 @@ export const buildTree = (
   ) {
     const currentTreeItem = {
       key: node.id,
-      title: node.meta.sliceName || node.meta.text || node.id.toString(),
+      title:
+        node.meta.sliceNameOverride ||
+        node.meta.sliceName ||
+        node.meta.text ||
+        node.id.toString(),
       children: [],
     };
     treeItem.children.push(currentTreeItem);
diff --git a/superset-frontend/src/dashboard/types.ts 
b/superset-frontend/src/dashboard/types.ts
index 9a15b1d..bd6eeed 100644
--- a/superset-frontend/src/dashboard/types.ts
+++ b/superset-frontend/src/dashboard/types.ts
@@ -93,6 +93,7 @@ export type LayoutItem = {
     chartId: number;
     height: number;
     sliceName?: string;
+    sliceNameOverride?: string;
     text?: string;
     uuid: string;
     width: number;

Reply via email to