Skip to content

Conversation

@jscheffl
Copy link
Contributor

@jscheffl jscheffl commented Aug 27, 2025

Following #53563 this PR enables auto-refresh.

@boring-cyborg boring-cyborg bot added area:dev-tools area:providers backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch kind:documentation provider:edge Edge Executor / Worker (AIP-69) / edge3 labels Aug 27, 2025
@jscheffl jscheffl force-pushed the feature/add-auto-refresh-to-edge-react-ui branch from 7d266eb to c95be76 Compare September 3, 2025 13:18
@jscheffl jscheffl removed area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Sep 3, 2025
@jscheffl jscheffl marked this pull request as ready for review September 3, 2025 13:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements auto-refresh functionality for the Edge React UI by adding a 5-second refetch interval to data queries. The change removes the auto-refresh item from TODO lists as it has now been implemented.

  • Adds auto-refresh with 5-second intervals to both worker and jobs pages
  • Updates hash file to reflect the UI changes
  • Removes completed auto-refresh TODO items from comments

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
providers/edge3/www-hash.txt Updates hash to reflect UI bundle changes
providers/edge3/src/airflow/providers/edge3/plugins/www/src/pages/WorkerPage.tsx Enables auto-refresh for worker data with 5-second interval
providers/edge3/src/airflow/providers/edge3/plugins/www/src/pages/JobsPage.tsx Enables auto-refresh for jobs data with 5-second interval

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small nits with should make the code cleaner. looking good otherwise.

@jscheffl jscheffl merged commit 1e406ab into apache:main Sep 3, 2025
59 checks passed
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Sep 30, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 1, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 2, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 3, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 4, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 7, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 8, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 9, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 10, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 11, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 12, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 14, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 15, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 17, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 19, 2025
* Add auto refresh to Edge Worker react UI

* Review feedback Copilot

* Review feedback Pierre
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers kind:documentation provider:edge Edge Executor / Worker (AIP-69) / edge3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants