kyungjunleeme commented on code in PR #53597:
URL: https://github.com/apache/airflow/pull/53597#discussion_r2229871044
##########
airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/openapi-gen/queries/queries.ts:
##########
@@ -6,11 +6,7 @@ import { LoginBody } from "../requests/types.gen";
import * as Common from "./common";
export const useSimpleAuthManagerLoginServiceCreateTokenAllAdmins = <TData =
Common.SimpleAuthManagerLoginServiceCreateTokenAllAdminsDefaultResponse, TError
= unknown, TQueryKey extends Array<unknown> = unknown[]>(queryKey?: TQueryKey,
options?: Omit<UseQueryOptions<TData, TError>, "queryKey" | "queryFn">) =>
useQuery<TData, TError>({ queryKey:
Common.UseSimpleAuthManagerLoginServiceCreateTokenAllAdminsKeyFn(queryKey),
queryFn: () => SimpleAuthManagerLoginService.createTokenAllAdmins() as TData,
...options });
export const useSimpleAuthManagerLoginServiceLoginAllAdmins = <TData =
Common.SimpleAuthManagerLoginServiceLoginAllAdminsDefaultResponse, TError =
unknown, TQueryKey extends Array<unknown> = unknown[]>(queryKey?: TQueryKey,
options?: Omit<UseQueryOptions<TData, TError>, "queryKey" | "queryFn">) =>
useQuery<TData, TError>({ queryKey:
Common.UseSimpleAuthManagerLoginServiceLoginAllAdminsKeyFn(queryKey), queryFn:
() => SimpleAuthManagerLoginService.loginAllAdmins() as TData, ...options });
-export const useSimpleAuthManagerLoginServiceCreateToken = <TData =
Common.SimpleAuthManagerLoginServiceCreateTokenMutationResult, TError =
unknown, TContext = unknown>(options?: Omit<UseMutationOptions<TData, TError, {
- requestBody: LoginBody;
-}, TContext>, "mutationFn">) => useMutation<TData, TError, {
- requestBody: LoginBody;
-}, TContext>({ mutationFn: ({ requestBody }) =>
SimpleAuthManagerLoginService.createToken({ requestBody }) as unknown as
Promise<TData>, ...options });
+export const useSimpleAuthManagerLoginServiceCreateToken = <TData =
Common.SimpleAuthManagerLoginServiceCreateTokenMutationResult, TError =
unknown, TContext = unknown>(options?: Omit<UseMutationOptions<TData, TError,
void, TContext>, "mutationFn">) => useMutation<TData, TError, void, TContext>({
mutationFn: () => SimpleAuthManagerLoginService.createToken() as unknown as
Promise<TData>, ...options });
Review Comment:
1)
1)
Due to the issue addressed in
[#53636](https://github.com/apache/airflow/pull/53636), a chain of operations
was not functioning properly.
2)
I didn’t want to apply any manual fixes.
I also don’t prefer making manual modifications myself, but since I’m
handling requests through a utility function, I believe this part is
unfortunately necessary.
3)
Additionally, it seems that even in the local environment, the part that was
skipped by pre-commit is causing issues.
Locally, pre-commit doesn’t run with the --all-files option, but in the CI
environment, it runs on all files, which causes static check errors.
But this case
Thank you for checking.
--
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]
