anshuksi282-ksolves commented on PR #59231: URL: https://github.com/apache/airflow/pull/59231#issuecomment-3682019460
Thanks for the continued review! I have successfully rebased with main and resolved the conflicts regarding the new UI structure (VStack & DataInterval fields). **Refactor Update:** I noticed that TriggerDAGForm.tsx was exceeding the 250-line limit linting rule after the merge. To adhere to clean code standards, I extracted the business logic into a dedicated utility file: * `src/utils/trigger.ts`: Moved all helper functions (e.g., `mergeUrlParams, extractParamValues, getTriggerConf`) and Type definitions here. * `TriggerDAGForm.tsx`: Now focuses solely on the UI and Form state, keeping the component lightweight and readable. * Tested locally, and the URL parameter syncing is working perfectly with the new UI layout. Ready for a final look! -- 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]
