You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by ma...@apache.org on 2025/08/20 06:54:21 UTC
(superset) 07/08: fix: Include drill_through_chart_id in dataset save payload
This is an automated email from the ASF dual-hosted git repository.
maximebeauchemin pushed a commit to branch custom-drillthrough
in repository https://gitbox.apache.org/repos/asf/superset.git
commit ddea28b6e477b835089261b00ac366feca93c77a
Author: Maxime Beauchemin <ma...@gmail.com>
AuthorDate: Mon Aug 11 16:50:43 2025 -0400
fix: Include drill_through_chart_id in dataset save payload
The drill_through_chart_id field was not being included in the payload when saving
a dataset in DatasourceModal, causing the selected chart to not persist. This fix
adds the field to the buildPayload function.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <no...@anthropic.com>
---
superset-frontend/src/components/Datasource/DatasourceModal.tsx | 1 +
1 file changed, 1 insertion(+)
diff --git a/superset-frontend/src/components/Datasource/DatasourceModal.tsx b/superset-frontend/src/components/Datasource/DatasourceModal.tsx
index 6e551a9a15..38a1d6e93e 100644
--- a/superset-frontend/src/components/Datasource/DatasourceModal.tsx
+++ b/superset-frontend/src/components/Datasource/DatasourceModal.tsx
@@ -179,6 +179,7 @@ const DatasourceModal: FunctionComponent<DatasourceModalProps> = ({
owners: datasource.owners.map(
(o: Record<string, number>) => o.value || o.id,
),
+ drill_through_chart_id: datasource.drill_through_chart_id || null,
};
// Handle catalog based on database's allow_multi_catalog setting
// If multi-catalog is disabled, don't include catalog in payload