entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 9 Releases Wiki Activity

api: add end_datetime in event details (#60789) #48

Merged
ecazenave merged 2 commits from wip/60789-end-datetime into main 2023-03-13 17:42:59 +01:00
Conversation 0 Commits 2 Files Changed 3 +55
ecazenave commented 2023-03-09 12:02:43 +01:00
Owner
Copy Link
No description provided.
ecazenave force-pushed wip/60789-end-datetime from 80f5420f78 to 55cae49305 2023-03-09 14:53:57 +01:00 Compare
ecazenave changed title from WIP: api: add end_datetime in event details (#60789) to api: add end_datetime in event details (#60789) 2023-03-09 15:00:50 +01:00
lguerin reviewed 2023-03-13 12:13:34 +01:00
chrono/api/views.py
@ -486,1 +486,4 @@
agenda = agenda or event.agenda
end_datetime = event.start_datetime
if event.duration:
end_datetime = event.start_datetime + datetime.timedelta(minutes=event.duration)
lguerin commented 2023-03-13 12:12:57 +01:00
Owner
Copy Link

il y a une property end_datetime sur Event, pourquoi ne pas l'utiliser ?

il y a une property end_datetime sur Event, pourquoi ne pas l'utiliser ?
ecazenave commented 2023-03-13 15:10:00 +01:00
Author
Owner
Copy Link

Effectivement c'était n'importe quoi mon truc, voilà avec event.end_datetime.

Avec tout de même la subtilité que sur un évènement sans durée event.end_datetime vaut None et format_response_datetime(None) sort un datetime.now hors sujet, donc dans ce cas j'opte pour une chaîne de caractères vide.

Effectivement c'était n'importe quoi mon truc, voilà avec event.end_datetime. Avec tout de même la subtilité que sur un évènement sans durée event.end_datetime vaut None et format_response_datetime(None) sort un datetime.now hors sujet, donc dans ce cas j'opte pour une chaîne de caractères vide.
ecazenave force-pushed wip/60789-end-datetime from 55cae49305 to 885f29cf24 2023-03-13 12:53:02 +01:00 Compare
ecazenave force-pushed wip/60789-end-datetime from 885f29cf24 to 46e7b51357 2023-03-13 13:19:39 +01:00 Compare
lguerin approved these changes 2023-03-13 17:40:42 +01:00
ecazenave merged commit 39523e4e65 into main 2023-03-13 17:42:59 +01:00
ecazenave deleted branch wip/60789-end-datetime 2023-03-13 17:42:59 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/chrono#48
No description provided.
Delete Branch "wip/60789-end-datetime"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?