shashbha14 commented on code in PR #67253:
URL: https://github.com/apache/airflow/pull/67253#discussion_r3281892542
##########
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:
Added a hasAppliedPrefill flag ,the second useEffect is now blocked until
the prefill has been applied once, then allows subsequent param changes to
sync through. This respects the prefill on initial load while letting user
edits take effect on submit.
--
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]
