jason810496 commented on code in PR #54597:
URL: https://github.com/apache/airflow/pull/54597#discussion_r2327728390
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/hitl.py:
##########
@@ -64,7 +64,7 @@ def _get_task_instance(
dag_run_id: str,
task_id: str,
session: SessionDep,
- map_index: int,
+ map_index: int | None,
Review Comment:
This one is not reverted yet as well.
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/hitl.py:
##########
@@ -131,15 +132,31 @@ def _update_hitl_detail(
if user_id not in hitl_detail_model.respondents:
log.error("User=%s (id=%s) is not a respondent for the task",
user_name, user_id)
raise HTTPException(
- status.HTTP_403_FORBIDDEN,
- f"User={user_name} (id={user_id}) is not a respondent for the
task.",
+ status_code=status.HTTP_403_FORBIDDEN,
Review Comment:
This change also seems not related as well.
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
