Skip to content

Commit 48ffd65

Browse files
committed
Fix useLogs after review
1 parent 80278f1 commit 48ffd65

File tree

3 files changed

+3
-16
lines changed

3 files changed

+3
-16
lines changed

airflow-core/src/airflow/ui/src/pages/Dag/Overview/TaskLogPreview.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
* under the License.
1818
*/
1919
import { Box, Flex, Link } from "@chakra-ui/react";
20-
import { useTranslation } from "react-i18next";
2120
import { Link as RouterLink } from "react-router-dom";
2221

2322
import type { TaskInstanceResponse } from "openapi/requests/types.gen";
@@ -35,13 +34,11 @@ export const TaskLogPreview = ({
3534
readonly taskInstance: TaskInstanceResponse;
3635
readonly wrap: boolean;
3736
}) => {
38-
const { t: translate } = useTranslation();
3937
const { data, error, isLoading } = useLogs(
4038
{
4139
dagId: taskInstance.dag_id,
4240
logLevelFilters: ["error", "critical"],
4341
taskInstance,
44-
translate,
4542
tryNumber: taskInstance.try_number,
4643
},
4744
{

airflow-core/src/airflow/ui/src/pages/TaskInstance/Logs/Logs.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import { Box, Heading, VStack } from "@chakra-ui/react";
2020
import { useState } from "react";
2121
import { useHotkeys } from "react-hotkeys-hook";
22-
import { useTranslation } from "react-i18next";
2322
import { useParams, useSearchParams } from "react-router-dom";
2423

2524
import { useTaskInstanceServiceGetMappedTaskInstance } from "openapi/queries";
@@ -33,7 +32,6 @@ import { TaskLogContent } from "./TaskLogContent";
3332
import { TaskLogHeader } from "./TaskLogHeader";
3433

3534
export const Logs = () => {
36-
const { t: translate } = useTranslation();
3735
const { dagId = "", mapIndex = "-1", runId = "", taskId = "" } = useParams();
3836
const [searchParams, setSearchParams] = useSearchParams();
3937

@@ -87,7 +85,6 @@ export const Logs = () => {
8785
logLevelFilters,
8886
sourceFilters,
8987
taskInstance,
90-
translate,
9188
tryNumber: tryNumber === 0 ? 1 : tryNumber,
9289
});
9390

airflow-core/src/airflow/ui/src/queries/useLogs.tsx

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ import { chakra } from "@chakra-ui/react";
2020
import type { UseQueryOptions } from "@tanstack/react-query";
2121
import dayjs from "dayjs";
2222
import type { TFunction } from "i18next";
23+
import { useTranslation } from "react-i18next";
2324
import innerText from "react-innertext";
2425

2526
import { useTaskInstanceServiceGetLog } from "openapi/queries";
@@ -34,7 +35,6 @@ type Props = {
3435
logLevelFilters?: Array<string>;
3536
sourceFilters?: Array<string>;
3637
taskInstance?: TaskInstanceResponse;
37-
translate: TFunction;
3838
tryNumber?: number;
3939
};
4040

@@ -138,17 +138,10 @@ const parseLogs = ({
138138
};
139139

140140
export const useLogs = (
141-
{
142-
accept = "application/json",
143-
dagId,
144-
logLevelFilters,
145-
sourceFilters,
146-
taskInstance,
147-
translate,
148-
tryNumber = 1,
149-
}: Props,
141+
{ accept = "application/json", dagId, logLevelFilters, sourceFilters, taskInstance, tryNumber = 1 }: Props,
150142
options?: Omit<UseQueryOptions<TaskInstancesLogResponse>, "queryFn" | "queryKey">,
151143
) => {
144+
const { t: translate } = useTranslation("common");
152145
const refetchInterval = useAutoRefresh({ dagId });
153146

154147
const { data, ...rest } = useTaskInstanceServiceGetLog(

0 commit comments

Comments
 (0)