Re: [PR] AIP-38 Add logical date to trigger DagRun form [airflow]
pierrejeambrun commented on PR #46471: URL: https://github.com/apache/airflow/pull/46471#issuecomment-2651345598 > Can we have some frontend magic to help user fill the fields since they are related to each other most of the time? Say > If data interval fields are empty, fill them with logical date if it’s filled > If logical date field is empty, fill it with data interval end or start if available (in this order) It appears that we want to completely get rid of the `dataIntervalStart` and `dataIntervalEnd` in the trigger form. Updated accordingly in my last commit. cc: @cmarteepants -- 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]
Re: [PR] AIP-38 Add logical date to trigger DagRun form [airflow]
pierrejeambrun commented on code in PR #46471:
URL: https://github.com/apache/airflow/pull/46471#discussion_r1950699853
##
airflow/ui/src/queries/useTrigger.ts:
##
@@ -73,6 +73,8 @@ export const useTrigger = ({ dagId, onSuccessConfirm }: {
dagId: string; onSucce
? new Date(dagRunRequestBody.dataIntervalEnd)
: undefined;
+const LogicalDate = dagRunRequestBody.logicalDate ? new
Date(dagRunRequestBody.logicalDate) : undefined;
Review Comment:
Done
--
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]
Re: [PR] AIP-38 Add logical date to trigger DagRun form [airflow]
uranusjr commented on PR #46471: URL: https://github.com/apache/airflow/pull/46471#issuecomment-2647213300 Can we have some frontend magic to help user fill the fields since they are related to each other most of the time? Say * If data interval fields are empty, fill them with logical date if it’s filled * If logical date field is empty, fill it with data interval end or start if available (in this order) -- 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]
Re: [PR] AIP-38 Add logical date to trigger DagRun form [airflow]
pierrejeambrun commented on code in PR #46471:
URL: https://github.com/apache/airflow/pull/46471#discussion_r1944916543
##
airflow/ui/src/queries/useTrigger.ts:
##
@@ -73,6 +73,8 @@ export const useTrigger = ({ dagId, onSuccessConfirm }: {
dagId: string; onSucce
? new Date(dagRunRequestBody.dataIntervalEnd)
: undefined;
+const LogicalDate = dagRunRequestBody.logicalDate ? new
Date(dagRunRequestBody.logicalDate) : undefined;
Review Comment:
ofc I'll do that. True that's weird :p I naively followed the convention,
but lower case is more appropriate :)
--
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]
Re: [PR] AIP-38 Add logical date to trigger DagRun form [airflow]
bbovenzi commented on code in PR #46471:
URL: https://github.com/apache/airflow/pull/46471#discussion_r1943424042
##
airflow/ui/src/queries/useTrigger.ts:
##
@@ -73,6 +73,8 @@ export const useTrigger = ({ dagId, onSuccessConfirm }: {
dagId: string; onSucce
? new Date(dagRunRequestBody.dataIntervalEnd)
: undefined;
+const LogicalDate = dagRunRequestBody.logicalDate ? new
Date(dagRunRequestBody.logicalDate) : undefined;
Review Comment:
Oh I didn't catch this before but all of these variables should start with a
lower case letter.
Do you mind fixing it here and for dataIntervalStart and dataIntervalEnd?
--
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]
