tirkarthi commented on PR #46820: URL: https://github.com/apache/airflow/pull/46820#issuecomment-2667706827
> Airflow 2 also went over the logs line-by-line. Don't know if you checked the old implementation. Do you think this could be factored-in? Thanks, I checked it. From what I understand in Airflow 2 manually created span tags appending to each other and used `dangerouslySetInnerHTML` to set the manually created html to div. This is simpler and doesn't need that. There should be a way to take nested groups through recursion but I cannot get the correct approach. I would be okay for nested implementation to be added in another PR to not block this PR. -- 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]
