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