github-actions[bot] opened a new pull request, #60225:
URL: https://github.com/apache/airflow/pull/60225

   * fix: Enable real-time extra links updates for TriggerDagRunOperator
   
   - Add smart polling to ExtraLinks component using useAutoRefresh hook
   - Poll automatically when DAG is active and links are not yet available
   - Stop polling once extra links appear to reduce server load
   - Respects global auto_refresh_interval config and DAG paused state
   - Fixes issue where Triggered DAG button only appeared after manual refresh
   
   Closes #58928
   
   * fix: prevent infinite polling for tasks without extra links
   
   - Add task metadata check using useTaskServiceGetTask
   - Only poll if task.extra_links.length > 0
   - Conditionally render ExtraLinks component in Details.tsx
   - Prevents unnecessary API calls for tasks without extra links
   
   Addresses review feedback from @bbovenzi
   
   * refactor: reuse parent refetchInterval pattern in child components
   
   - Pass parent's refetchInterval to ExtraLinks and BlockingDeps components
   - Remove duplicate useAutoRefresh and useTaskServiceGetTask calls
   - Eliminate invalid taskInstance.extra_links property access (doesn't exist 
on TaskInstanceResponse)
   - Polling now tied to try instance pending state - stops automatically when 
task completes
   - Simplifies architecture by centralizing refresh logic in parent component
   
   Addresses review feedback from @pierrejeambrun and @bbovenzi
   
   * style: apply pre-commit hook formatting fixes
   
   - Reorder type union to alphabetical order (number | false)
   - Format ExtraLinks component to single line
   (cherry picked from commit 2768fba69549cb4030af8ce97c675ef893433017)
   
   Co-authored-by: subhash-0000 
<[email protected]>


-- 
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]

Reply via email to