Lee-W commented on PR #43073: URL: https://github.com/apache/airflow/pull/43073#issuecomment-2428014469
> Mhm, I am not good with this PR. We will "throw away" the old UI once we have the New ready. Until then we need to maintain old and new UI. > > If we now mass-rename the legacy UI parts for Dataset->Asset... the legacy UI wiull diverge a lot from the 2.x line. This means any patch/contribution would need to be adjusted not to "by accident" convert Dataset to Asset in the 2.x line. > > Could we leave the legacy UI un-touched and rename just the new UI? because: > > 1. Maintenance on the legacy will be harder if the legacy diverges > > 2. Is is a un-needed change if the UI anyway will be removed (soon) I'm unsure 🤔 The concept of datasets no longer exists in the main branch, so keeping these dataset things in the main branch seems odd. I'm uncertain whether we anticipate many changes to the UI side in version 2.x, but even if so, those changes could probably go to the 2.x branch. But I'm not heavily contributing to the UI end of Airflow, so I would love to know what @bbovenzi thinks. -- 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]
