This is an automated email from the ASF dual-hosted git repository.
beto pushed a commit to branch folder-api
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/folder-api by this push:
new 06f50d074a Remove folders from response
06f50d074a is described below
commit 06f50d074a9fe88271c583eff600922931b68304
Author: Beto Dealmeida <[email protected]>
AuthorDate: Thu Mar 27 18:21:59 2025 -0400
Remove folders from response
---
superset/datasets/api.py | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/superset/datasets/api.py b/superset/datasets/api.py
index 7851d5a34c..85cc075667 100644
--- a/superset/datasets/api.py
+++ b/superset/datasets/api.py
@@ -35,7 +35,7 @@ from flask_babel import ngettext
from jinja2.exceptions import TemplateSyntaxError
from marshmallow import ValidationError
-from superset import event_logger
+from superset import event_logger, is_feature_enabled
from superset.commands.dataset.create import CreateDatasetCommand
from superset.commands.dataset.delete import DeleteDatasetCommand
from superset.commands.dataset.duplicate import DuplicateDatasetCommand
@@ -1156,6 +1156,14 @@ class DatasetRestApi(BaseSupersetModelRestApi):
response["id"] = pk
response[API_RESULT_RES_KEY] = show_model_schema.dump(item, many=False)
+ # remove folders from resposne if `DATASET_FOLDERS` is disabled, so
that it's
+ # possible to inspect if the feature is supported or not
+ if (
+ not is_feature_enabled("DATASET_FOLDERS")
+ and "folders" in response[API_RESULT_RES_KEY]
+ ):
+ del response[API_RESULT_RES_KEY]["folders"]
+
if parse_boolean_string(request.args.get("include_rendered_sql")):
try:
processor = get_template_processor(database=item.database)
