pierrejeambrun commented on code in PR #35403:
URL: https://github.com/apache/airflow/pull/35403#discussion_r1410283905
##########
airflow/www/static/js/api/useGridData.ts:
##########
@@ -105,6 +105,7 @@ const useGridData = () => {
};
const response = await axios.get<AxiosResponse, GridData>(gridDataUrl, {
params,
+ paramsSerializer: { indexes: null },
Review Comment:
The extract is missing some axios calls, for instance in the
useClearTaskDryRun, it is cut off too soon, we don’t have the axios call I
believe.
But that would be the idea checking in those files if an array is used as
query param.
I think we are good.
Also we have a few tests in the UI and you can manually do some UI testing
to see if anything is broken after implementing the global axios config change
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
