api: hide check fields for bookings of meetings agendas (#69169) #3
No reviewers
Labels
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: entrouvert/chrono#3
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/69169-api-check-fields-for-meetings"
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?
api: hide check fields for bookings of meetings agendas (#69169)to WIP: api: hide check fields for bookings of meetings agendas (#69169)WIP: api: hide check fields for bookings of meetings agendas (#69169)to api: hide check fields for bookings of meetings agendas (#69169)Tout est bon pour moi a part les deux remarques.
@ -237,1 +232,3 @@
)
if self.instance.event.agenda.kind != 'events':
ret.pop('user_was_present', None)
ret.pop('user_absence_reason', None)
Pourquoi on a besoin de ça ici ?
@aberriot c'est pour une raison X.
@ -2540,0 +2543,4 @@
or 'user_absence_reason' in request.data
or 'user_presence_reason' in request.data
):
raise APIErrorBadRequest(N_('can not set check fields for non events agenda'), err=7)
Je crois qu'il y a une typo.
Un petit test
Ok pour moi.
(relu pour de vrai, je revalide à posteriori)