bbovenzi commented on code in PR #67253:
URL: https://github.com/apache/airflow/pull/67253#discussion_r3276958263
##########
airflow-core/src/airflow/ui/src/components/TriggerDag/TriggerDAGForm.tsx:
##########
@@ -130,12 +126,12 @@ const TriggerDAGForm = ({
isPartitioned,
]);
- // Automatically reset form when conf is fetched (only if no prefillConfig)
+ // Automatically reset form when conf is fetched
useEffect(() => {
- if (conf && open && (!prefillConfig || hasAppliedPrefill)) {
+ if (conf && open) {
Review Comment:
We can't just delete prefillConfig. Please try to fix this while respecting
the prefill. I suspect that we need to tackle the number of sources of truth we
have for run config and remove the need to sync so much of it.
--
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]
