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

api: events check endpoint, return also times (#80973) #140

Merged
lguerin merged 1 commits from wip/80973-api-calculated-period into main 2023-09-15 09:12:20 +02:00
Conversation 3 Commits 1 Files Changed 2 +210 -115
lguerin commented 2023-09-07 12:20:41 +02:00
Owner
Copy Link
No description provided.
lguerin changed title from WIP: api: events check endpoint, return also times (#80973) to api: events check endpoint, return also times (#80973) 2023-09-07 16:19:43 +02:00
lguerin force-pushed wip/80973-api-calculated-period from 631d042da0 to bd55c622ca 2023-09-07 16:22:31 +02:00 Compare
lguerin force-pushed wip/80973-api-calculated-period from bd55c622ca to 782e6ee4db 2023-09-08 08:39:04 +02:00 Compare
lguerin force-pushed wip/80973-api-calculated-period from 782e6ee4db to a179ae7a71 2023-09-08 12:10:07 +02:00 Compare
vdeniaud commented 2023-09-13 15:01:29 +02:00
Owner
Copy Link

J'ai tiqué sur les 9 pop successifs et j'ai l'impression qu'on peut s'en sortir avec un patch plus succin, j'attache ma tentative tapée vite fait mais qui passe le test.

Aussi éventuellement je trouverais ça plus joli de coller au nom des champs du modèle en retirant le préfixe « booked_ », ça donnerait « start_time », « end_time », et « minutes » tout seul ça ne sonne pas très bien, on pourrait parler de « duration », champ qui existe déjà pour qualifier la durée en minutes d'un évènement ?

J'ai tiqué sur les 9 `pop` successifs et j'ai l'impression qu'on peut s'en sortir avec un patch plus succin, j'attache ma tentative tapée vite fait mais qui passe le test. Aussi éventuellement je trouverais ça plus joli de coller au nom des champs du modèle en retirant le préfixe « booked_ », ça donnerait « start_time », « end_time », et « minutes » tout seul ça ne sonne pas très bien, on pourrait parler de « duration », champ qui existe déjà pour qualifier la durée en minutes d'un évènement ?
0001-test-less-lines.patch
4.2 KiB
lguerin force-pushed wip/80973-api-calculated-period from a179ae7a71 to fad838007f 2023-09-14 09:30:51 +02:00 Compare
lguerin force-pushed wip/80973-api-calculated-period from fad838007f to 4f86dbf687 2023-09-14 09:34:02 +02:00 Compare
lguerin commented 2023-09-14 09:34:24 +02:00
Author
Owner
Copy Link

patch intégré (merci !), et champs renommés.

patch intégré (merci !), et champs renommés.
nroche approved these changes 2023-09-14 10:02:38 +02:00
nroche left a comment
Owner
Copy Link

Ack

Ack
vdeniaud commented 2023-09-14 10:03:45 +02:00
Owner
Copy Link

Ack

(c'est moi qui ait demandé à Nico de valider, un bug qui m'empêchait de le faire)

> Ack (c'est moi qui ait demandé à Nico de valider, un bug qui m'empêchait de le faire)
lguerin changed title from api: events check endpoint, return also times (#80973) to api: events check endpoint, return also times (#80973) 2023-09-15 08:51:33 +02:00
lguerin changed target branch from wip/80842-partial-booking-invoiced-bookings to main 2023-09-15 08:51:34 +02:00
lguerin force-pushed wip/80973-api-calculated-period from 4f86dbf687 to c8d71aa997 2023-09-15 08:57:04 +02:00 Compare
lguerin merged commit c8d71aa997 into main 2023-09-15 09:12:20 +02:00
lguerin deleted branch wip/80973-api-calculated-period 2023-09-15 09:12:20 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#140
No description provided.
Delete Branch "wip/80973-api-calculated-period"

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?