[GitHub] [airflow] pierrejeambrun commented on pull request #25610: Grid logs for mapped instances

[GitBox <gi...@apache.org>]

pierrejeambrun commented on PR #25610:
URL: https://github.com/apache/airflow/pull/25610#issuecomment-1209793600

   I updated the PR and the description.
   
   - Nav link is done, and removed them from the table, nice suggestion. :)
   I'm not sure about direct log access, I feel like it is already covered by the preferred tab. If the user preferred tab is 'logs" he will directly land on logs by clicking on the table row otherwise he will land on details.
   
   - Added the button to go back to the dynamic task summary, good idea!
   
   - Separate the mapped task table on a new tab only available for mapped task summary. Unfortunately I do not have the number of mapped instance info available atm. (this could be an improvement).
   
   `Logs` & `Mapped Tasks` tab are mutually exclusive. In terms of 'preferredTab' they act as the same. Meaning you can go from 'mapped' -> 'logs' and reverse without changing tabs.
   
   


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org