github-actions[bot] opened a new pull request, #66482:
URL: https://github.com/apache/airflow/pull/66482
Fixes #66428.
The Required Actions listing component hard-coded
``mapIndex: parseInt(searchParams.get(MAP_INDEX) ?? "-1", 10)``, which
always sent ``map_index=-1`` and silently filtered every page (the global
``/required_actions`` view as well as the per-DAG / per-Run / per-Task
Required Actions tabs) down to non-mapped task instances. Mapped HITL
rows were therefore counted by the home-page badge but invisible in the
listing pages.
* Pass ``mapIndex`` to ``useTaskInstanceServiceGetHitlDetails`` only when
the user has explicitly set the ``map_index`` URL search param. The
default behaviour is now "all map indexes" (no filter) instead of
"non-mapped only", and ``map_index=-1`` continues to work as an opt-in
filter for non-mapped task instances.
* Fix the auto-refresh predicate that compared ``responded_at === undefined``
instead of ``=== null``. The API serialises ``responded_at`` as JSON
``null``, never an omitted field, so the predicate never matched and
the listing never polled for new pending actions.
* Add ``HITLTaskInstances.test.tsx`` covering the three ``map_index``
URL branches (absent / set to integer / set to ``-1``) and the refetch
predicate for both ``responded_at: null`` and a populated value.
(cherry picked from commit e3e3f6fad2b37ba054070ff3ebcff5296f57fc9d)
Co-authored-by: Paul Mathew <[email protected]>
Signed-off-by: Paul Mathew <[email protected]>
Co-authored-by: Cursor <[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]
