Skip to content

Front end for hitl endpoint update - #55674

Merged
kaxil merged 2 commits into
apache:mainfrom
astronomer:front-end-for-hitl-endpoint-update
Sep 15, 2025
Merged

Front end for hitl endpoint update#55674
kaxil merged 2 commits into
apache:mainfrom
astronomer:front-end-for-hitl-endpoint-update

Conversation

@pierrejeambrun

@pierrejeambrunpierrejeambrun commented Sep 15, 2025

Copy link
Copy Markdown
Member

Follow up of #55603.

Only the last commit is relevant. Tested locally all seems good.

@boring-cyborgboring-cyborgBot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Sep 15, 2025
@pierrejeambrunpierrejeambrun added this to the Airflow 3.1.0 milestone Sep 15, 2025
jason810496
jason810496 approved these changes Sep 15, 2025
View reviewed changes

@jason810496jason810496 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice! LGTM for the change.

Comment threadairflow-core/src/airflow/api_fastapi/core_api/routes/public/hitl.py
bbovenzi
bbovenzi approved these changes Sep 15, 2025
View reviewed changes

@bbovenzibbovenzi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like we need to merge this PR with both commits to avoid breaking the UI

@kaxil
kaxil merged commit c14c18e into apache:mainSep 15, 2025
111 checks passed
@kaxil
kaxil deleted the front-end-for-hitl-endpoint-update branch September 15, 2025 21:32
kaxil pushed a commit that referenced this pull request Sep 15, 2025
@pierrejeambrun@kaxil
Front end for hitl endpoint update (#55674)
ba0f359
Follow up of #55603.
Only the last commit is relevant. Tested locally all seems good.
(cherry picked from commit c14c18e)
kaxil pushed a commit that referenced this pull request Sep 15, 2025
@pierrejeambrun@kaxil
Front end for hitl endpoint update (#55674)
f6377f6
Follow up of #55603.
Only the last commit is relevant. Tested locally all seems good.
(cherry picked from commit c14c18e)
yash1thsa pushed a commit to yash1thsa/airflow that referenced this pull request Sep 16, 2025
@pierrejeambrun@yash1thsa
Front end for hitl endpoint update (apache#55674)
f3e2645
Follow up of apache#55603.
Only the last commit is relevant. Tested locally all seems good.
suman-himanshu pushed a commit to suman-himanshu/airflow that referenced this pull request Sep 17, 2025
@pierrejeambrun@suman-himanshu
Front end for hitl endpoint update (apache#55674)
38b65b5
Follow up of apache#55603.
Only the last commit is relevant. Tested locally all seems good.
@potiukpotiuk mentioned this pull request Nov 8, 2025
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbovenzibbovenzibbovenzi approved these changes
@bugraoz93bugraoz93bugraoz93 approved these changes
@jason810496jason810496jason810496 approved these changes
@ryanahamiltonryanahamiltonAwaiting requested review from ryanahamiltonryanahamilton is a code owner
@jscheffljschefflAwaiting requested review from jscheffl
@shubhamraj-gitshubhamraj-gitAwaiting requested review from shubhamraj-gitshubhamraj-git is a code owner
@ephraimbuddyephraimbuddyAwaiting requested review from ephraimbuddyephraimbuddy is a code owner
@rawwarrawwarAwaiting requested review from rawwarrawwar is a code owner

Assignees

No one assigned

Labels

area:APIAirflow's REST/HTTP APIarea:UIRelated to UI/UX. For Frontend Developers.

Projects

None yet

Milestone

Airflow 3.1.0

Development

Successfully merging this pull request may close these issues.

6 participants

@pierrejeambrun@bbovenzi@bugraoz93@jason810496@kaxil@Lee-W