mariana-marcal-santana commented on code in PR #47849:
URL: https://github.com/apache/airflow/pull/47849#discussion_r2084274119
##########
airflow-core/src/airflow/ui/src/utils/query.ts:
##########
@@ -20,6 +20,11 @@ import { useDagServiceGetDagDetails } from "openapi/queries";
import type { TaskInstanceState } from "openapi/requests/types.gen";
import { useConfig } from "src/queries/useConfig";
+export const getOrderBy = (pre_defined?: string): string[] => {
+ const sortParam = new
URLSearchParams(globalThis.location.search).getAll("sort");
+ return (sortParam.length === 0 && pre_defined != undefined) ? [pre_defined]
: sortParam;
+};
+
Review Comment:
Hello @bbovenzi !
I had previously ran these operations and noticed that the first one changes
the v1-rest-api-generated.yaml file like so:
This is what is causing the errors in the CI because the Order by argument
now has to be an array and it keeps changing to string (which doesn't go along
with the changes in the code for the multisort).
Given this, what I meant was if there was a way to define the file and the
pre-commit to allow the Order By to be an array? From what I know, it seems
like this could fix the CI errors.
Thank you!
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
