Skip to content

Commit

Permalink
fix: remove feature flag for tags on Dashboard API for Tags (#23909)
Browse files Browse the repository at this point in the history
  • Loading branch information
hughhhh authored May 2, 2023
1 parent a8e02cd commit 3528f41
Showing 1 changed file with 14 additions and 26 deletions.
40 changes: 14 additions & 26 deletions superset/dashboards/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -191,9 +191,11 @@ def ensure_thumbnails_enabled(self) -> Optional[Response]:
"roles.id",
"roles.name",
"is_managed_externally",
"tags.id",
"tags.name",
"tags.type",
]
if is_feature_enabled("TAGGING_SYSTEM"):
list_columns += ["tags.id", "tags.name", "tags.type"]

list_select_columns = list_columns + ["changed_on", "created_on", "changed_by_fk"]
order_columns = [
"changed_by.first_name",
Expand All @@ -219,36 +221,22 @@ def ensure_thumbnails_enabled(self) -> Optional[Response]:
edit_columns = add_columns

search_columns = (
(
"created_by",
"changed_by",
"dashboard_title",
"id",
"owners",
"published",
"roles",
"slug",
"tags",
)
if is_feature_enabled("TAGGING_SYSTEM")
else (
"created_by",
"changed_by",
"dashboard_title",
"id",
"owners",
"published",
"roles",
"slug",
)
"created_by",
"changed_by",
"dashboard_title",
"id",
"owners",
"published",
"roles",
"slug",
"tags",
)
search_filters = {
"dashboard_title": [DashboardTitleOrSlugFilter],
"id": [DashboardFavoriteFilter, DashboardCertifiedFilter],
"created_by": [DashboardCreatedByMeFilter, DashboardHasCreatedByFilter],
"tags": [DashboardTagFilter],
}
if is_feature_enabled("TAGGING_SYSTEM"):
search_filters["tags"] = [DashboardTagFilter]

base_order = ("changed_on", "desc")

Expand Down

0 comments on commit 3528f41

Please sign in to comment.