QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-05-12 19:35:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(explore): Remove query autotrigger (#35418)

Browse Source
This commit is contained in:
Kamil Gabryjelski
2025-10-02 19:08:31 +02:00
committed by GitHub
parent 3202ff4b3f
commit 0364933e8b
2 changed files with 2 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
superset-frontend/src/explore/components/ExploreViewContainer/index.jsx
View File

@@ -371,15 +371,6 @@ function ExploreViewContainer(props) {
props.form_data,
]);
// Simple debounced auto-query for non-renderTrigger controls
const debouncedAutoQuery = useMemo(
() =>
debounce(() => {
onQuery();
}, 1000), // 1 second delay
[onQuery],
);
const handleKeydown = useCallback(
event => {
const controlOrCommand = event.ctrlKey || event.metaKey;
@@ -573,25 +564,8 @@ function ExploreViewContainer(props) {
if (displayControlsChanged.length > 0) {
reRenderChart(displayControlsChanged);
}
// Auto-update for non-renderTrigger controls
const queryControlsChanged = changedControlKeys.filter(
key =>
!props.controls[key].renderTrigger &&
!props.controls[key].dontRefreshOnChange,
);
if (queryControlsChanged.length > 0) {
// Check if there are no validation errors before auto-updating
const hasErrors = Object.values(props.controls).some(
control =>
control.validationErrors && control.validationErrors.length > 0,
);
if (!hasErrors) {
debouncedAutoQuery();
}
}
}
}, [props.controls, props.ownState, debouncedAutoQuery]);
}, [props.controls, props.ownState]);
const chartIsStale = useMemo(() => {
if (lastQueriedControls) {