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

manager: prefill presence check form with unexpected presence (#88039) #222

Merged
lguerin merged 1 commits from wip/88039-check-unexpected-presence into main 2024-03-15 08:43:20 +01:00
Conversation 0 Commits 1 Files Changed 5 +27 -2
lguerin commented 2024-03-12 16:44:55 +01:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2024-03-12 16:44:55 +01:00
vdeniaud approved these changes 2024-03-13 13:59:24 +01:00
chrono/manager/views.py Outdated
@ -1614,6 +1614,7 @@ class EventChecksMixin:
)
subscription.presence_form = BookingCheckPresenceForm(
agenda=self.agenda,
subscription=True,
vdeniaud commented 2024-03-13 13:58:02 +01:00
Owner
Copy Link

J'aurais trouvé ça un poil plus clair de passer initial=lebontruc ici plutôt que de construire dans le formulaire, ça serait aussi plus homogène avec ce qui est fait au dessus (la requête que fait get_agenda_check_types est mise en cache donc niveau perf j'imagine que ça ne changerait rien)

J'aurais trouvé ça un poil plus clair de passer initial=lebontruc ici plutôt que de construire dans le formulaire, ça serait aussi plus homogène avec ce qui est fait au dessus (la requête que fait get_agenda_check_types est mise en cache donc niveau perf j'imagine que ça ne changerait rien)
lguerin commented 2024-03-14 10:39:17 +01:00
Author
Owner
Copy Link

Moui, mais pour les tests c'est clairement pas pratique, il faudrait mocker deux fois, une fois dans forms et l'autre dans views (mocker chrono.utils.lingo.get_agenda_check_types ne fonctionne pas)

Moui, mais pour les tests c'est clairement pas pratique, il faudrait mocker deux fois, une fois dans forms et l'autre dans views (mocker chrono.utils.lingo.get_agenda_check_types ne fonctionne pas)
lguerin force-pushed wip/88039-check-unexpected-presence from b7cbff9b28 to e0f1d9541d 2024-03-15 08:38:19 +01:00 Compare
lguerin merged commit e0f1d9541d into main 2024-03-15 08:43:20 +01:00
lguerin deleted branch wip/88039-check-unexpected-presence 2024-03-15 08:43:20 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
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#222
No description provided.
Delete Branch "wip/88039-check-unexpected-presence"

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?