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

"préblocage" d'une réservation (#17685) #58

Closed
bdauvergne wants to merge 2 commits from wip/17685-preblocage-d-une-reservation into main
pull from: wip/17685-preblocage-d-une-reservation
merge into: entrouvert:main
Conversation 2 Commits 2 Files Changed 17 +1296 -30
bdauvergne commented 2023-03-19 19:42:04 +01:00
Owner
Copy Link
No description provided.
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from 815d3053a2 to b5d9bf35bb 2023-03-19 23:29:58 +01:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from b5d9bf35bb to 8693e72b4d 2023-03-20 00:13:14 +01:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from 8693e72b4d to caab16f6f7 2023-03-23 13:07:59 +01:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from caab16f6f7 to a5cbf29515 2023-03-23 13:55:48 +01:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from a5cbf29515 to 48e391709b 2023-03-23 14:13:34 +01:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from 48e391709b to e96e9b2c12 2023-03-23 14:24:17 +01:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from e96e9b2c12 to a0d96ddc0c 2023-03-23 14:29:19 +01:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from a0d96ddc0c to 32365f21b6 2023-03-23 14:38:05 +01:00 Compare
bdauvergne changed title from WIP: "préblocage" d'une réservation (#17685) to "préblocage" d'une réservation (#17685) 2023-03-23 14:42:34 +01:00
bdauvergne commented 2023-03-23 14:43:50 +01:00
Author
Owner
Copy Link

Les deux premiers patchs c'est pour faciliter l'écriture des tests, j'avais un peu de mal avec les fixture trucmuche_agenda, et les templates d'URL vers l'API un peu partout; comme ça c'est plus clair.

Les deux premiers patchs c'est pour faciliter l'écriture des tests, j'avais un peu de mal avec les fixture trucmuche_agenda, et les templates d'URL vers l'API un peu partout; comme ça c'est plus clair.
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from 32365f21b6 to 866d1bbec3 2023-03-23 14:49:27 +01:00 Compare
bdauvergne commented 2023-03-23 14:57:54 +01:00
Author
Owner
Copy Link

Pas tellement d'optimisations, sur la réservation des mêmes slots avec le même lock code on pourrait éviter de supprimer/recréer mais ça demanderait pas mal de logique pour pas grand chose.

Il faudrait que je vérifie si les clean_bookings_with_expired_leases() sont vraiment tous nécessaires, il me semble après un vague survol du code qu'un agenda utilisable dans certains endpoints (récurrent, souscription? je ne sais pas trop ce que ça couvre) ne l'est pas dans d'autres (et ici seuls les endpoints datetimes et fillslots classiques gèrent le lock_code). Si on m'indique à quoi ça sert/comment ça marche je peux voir pour y ajouter le support de lock_code. Je connais surtout le FillSlot classique.

Pas tellement d'optimisations, sur la réservation des mêmes slots avec le même lock code on pourrait éviter de supprimer/recréer mais ça demanderait pas mal de logique pour pas grand chose. Il faudrait que je vérifie si les clean_bookings_with_expired_leases() sont vraiment tous nécessaires, il me semble après un vague survol du code qu'un agenda utilisable dans certains endpoints (récurrent, souscription? je ne sais pas trop ce que ça couvre) ne l'est pas dans d'autres (et ici seuls les endpoints datetimes et fillslots classiques gèrent le lock_code). Si on m'indique à quoi ça sert/comment ça marche je peux voir pour y ajouter le support de lock_code. Je connais surtout le FillSlot classique.
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from 866d1bbec3 to 41d0f6f7ac 2023-04-21 13:28:29 +02:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from 41d0f6f7ac to 52f4c167cd 2023-05-02 12:27:12 +02:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from 52f4c167cd to f9bf8c6ce3 2023-05-02 13:43:58 +02:00 Compare
bdauvergne force-pushed wip/17685-preblocage-d-une-reservation from f9bf8c6ce3 to 34e866f178 2023-08-22 11:39:09 +02:00 Compare
bdauvergne closed this pull request 2023-08-22 12:45:01 +02:00
All checks were successful
gitea/chrono/pipeline/head This commit looks good
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 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#58
No description provided.
Delete Branch "wip/17685-preblocage-d-une-reservation"

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?