pierrejeambrun opened a new pull request, #25654:
URL: https://github.com/apache/airflow/pull/25654

   This PR just removes a statement that had no effect.
   
   We keep sending possibly 'None'  state in the grid data except for the 
'mapped_states' count. (None keys in json are an issue)
   
   I have another version of this PR 
[here](https://github.com/apache/airflow/compare/main...pierrejeambrun:airflow:fix-task-group-to-grid-no-status),
 where instead, None status are changed to 'no_status' in the entire returned 
payload. I don't think this is a good idea because either way the front will 
still have to handle None state. (Other endpoints returning `TaskInstance` will 
not do this transformation).
   
   I plan to work on `grid_data` endpoint api_connexion.TaskInstance. (run_id 
vs dag_run_id and other differences), to be able to unify those types in the 
front. (They are not playing nice with each other right now)


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