pierrejeambrun commented on code in PR #54994:
URL: https://github.com/apache/airflow/pull/54994#discussion_r2322263065
##########
providers/edge3/src/airflow/providers/edge3/plugins/www/src/pages/JobsPage.tsx:
##########
@@ -22,12 +22,17 @@ import { useUiServiceJobs } from "openapi/queries";
import { ErrorAlert } from "src/components/ErrorAlert";
export const JobsPage = () => {
- const { data, error } = useUiServiceJobs();
+ // Note: might use this in future but core-ui-API is not yet bound:
+ // const autoRefreshInterval = useConfig("auto_refresh_interval") as number
| undefined;
Review Comment:
> Else I'd prefer to leverage axios
ReactQuery on top of Axios I find is the most simplistic way to handle HTTP
calls. React Query is doing a lot on top of Axios to offer a more usable
interface. (exposing 'isSuccess', 'isLoading', 'isError', etc... on your calls,
doing some caching and retries automatically for us, etc...)
--
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]
