entrouvert
/
chrono
15
1
Fork 0
Code Issues Pull Requests 10 Releases Wiki Activity

callback sur les réservations lorsqu'un événement est marqué pointé (#75896) #63

Merged
fpeters merged 2 commits from wip/75896-callback-checked-events into main 2023-04-03 17:25:03 +02:00
Conversation 1 Commits 2 Files Changed 9
fpeters commented 2023-03-31 08:40:26 +02:00
Owner
There is no content yet.
fpeters added 2 commits 2023-03-31 08:40:26 +02:00
fpeters force-pushed wip/75896-callback-checked-events from 5b79d4ead9 to e772c57edd 2023-03-31 08:42:27 +02:00 Compare
fpeters force-pushed wip/75896-callback-checked-events from e772c57edd to 2ffaa13dc7 2023-03-31 08:46:25 +02:00 Compare
fpeters force-pushed wip/75896-callback-checked-events from 2ffaa13dc7 to 7844e8a37f 2023-03-31 09:13:59 +02:00 Compare
fpeters force-pushed wip/75896-callback-checked-events from 7844e8a37f to 37c3d91192 2023-04-03 14:52:28 +02:00 Compare
fpeters force-pushed wip/75896-callback-checked-events from 37c3d91192 to 2906c19f31 2023-04-03 15:18:05 +02:00 Compare
fpeters force-pushed wip/75896-callback-checked-events from 2906c19f31 to c377cb7cda 2023-04-03 15:51:14 +02:00 Compare
fpeters changed title from WIP: callback sur les réservations lorsqu'un événement est marqué pointé (#75896) to callback sur les réservations lorsqu'un événement est marqué pointé (#75896) 2023-04-03 15:57:01 +02:00
fpeters commented 2023-04-03 16:00:40 +02:00
Poster
Owner

Voilà c'est très très basique; pour la partie uwsgi j'ai fait comme l'existant et je n'ai pas testé en local (autant prévenir). Idéalement il y aurait sans doute toute une infra de rejeu des notifications mais j'ai trouvé ça excessif pour le moment, qu'en cas de soucis on se sauverait via les logs.

Voilà c'est très très basique; pour la partie uwsgi j'ai fait comme l'existant et je n'ai pas testé en local (autant prévenir). Idéalement il y aurait sans doute toute une infra de rejeu des notifications mais j'ai trouvé ça excessif pour le moment, qu'en cas de soucis on se sauverait via les logs.
lguerin reviewed 2023-04-03 16:06:42 +02:00
chrono/manager/views.py Outdated
@ -2748,6 +2748,7 @@ class EventCheckedView(EventCheckMixin, ViewableAgendaMixin, View):
if not self.event.checked:
self.event.checked = True
self.event.save(update_fields=['checked'])
self.event.async_notify_checked()
lguerin commented 2023-04-03 16:06:15 +02:00
Owner

on a aussi un endpoint pour marquer un event comme pointé, EventCheck

on a aussi un endpoint pour marquer un event comme pointé, EventCheck
fpeters commented 2023-04-03 17:13:58 +02:00
Poster
Owner

J'étais passé à côté, voilà il est traité, avec un test dédié.

(et c'est rebasé pour la migration)

J'étais passé à côté, voilà il est traité, avec un test dédié. (et c'est rebasé pour la migration)
lguerin marked this conversation as resolved
fpeters force-pushed wip/75896-callback-checked-events from c377cb7cda to ad2b04f89f 2023-04-03 16:41:24 +02:00 Compare
lguerin approved these changes 2023-04-03 17:11:07 +02:00
fpeters merged commit ad2b04f89f into main 2023-04-03 17:25:03 +02:00
fpeters deleted branch wip/75896-callback-checked-events 2023-04-03 17:25:03 +02: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#63
There is no content yet.
Delete Branch "wip/75896-callback-checked-events"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?