KyeongJooni commented on PR #59738: URL: https://github.com/apache/airflow/pull/59738#issuecomment-3687049291
Thanks for the review! I've updated the locators to scope them within the dialog context. I attempted to run the tests locally to verify the changes. The dialog scoping follows the same pattern used in DagsPage and ensures locators only match elements within the trigger dialog, preventing the strict mode violations. The key changes: - All form elements (backfillRadio, fromDateInput, toDateInput, runButton) are now scoped to `dialog.locator()` instead of `page.locator()` - This prevents conflicts with any duplicate elements that might exist elsewhere on the page Let me know if you'd like me to make any additional changes! -- 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]
