Produits Entr'ouvert » Chrono

Le ticket demande de pouvoir préciser une durée, pas une date/heure de fin.

Faire le ticket.

Et ensuite dans la propriété end_datetime, utiliser self.meeting_type.duration dans le cas rendez-vous et self.duration sinon, et fin.

Testé via wcs (pas sur la réservation multiple, j'avoue) :

Pour ne pas chercher à deviner ce qu'un bout de code signifie pour toi, tu peux expliciter, dire "ok c'est bon", "le patch m'a l'air ok mais pourtant la sortie n'est pas celle que j'attendais", etc. ?

-> ok c'est tout bon
(mais je n'ai pas testé dans wcs la réponse json sur les réservations multiples)

J'aurais fait un champ nullable sans default; là dans le cas d'un event de meeting, on aura une duration à 30 minutes (default) alors que le meeting_type aura probablement une autre duration.

Est-ce que la duration est optionnelle ou required dans le cas d'un event d'events ? Je pense qu'il serait mieux de gérer ça côté form que côté model.

    @property
    def end_datetime(self):
        if self.agenda.accept_meetings():
            minutes = self.meeting_type.duration
        else:
            minutes = self.duration
        return self.start_datetime + datetime.timedelta(minutes=minutes)

meeting_type est nullable et setté uniquement pour un event de meeting; ça serait peut-être mieux de baser la condition sur self.meeting_type is not None et non sur self.agenda (ça éviterait un potentiel queryset supplémentaire)

Il n'y a pas besoin de modifier la méthode get_ics ?

        if self.event.meeting_type:
            vevent.add('dtend').value = self.event.start_datetime + datetime.timedelta(
                minutes=self.event.meeting_type.duration
            )

Oui nécessairement optionnelle. (mais je ne vois pas l'enchainement dans la réflexion qui amène de get_ics à cette question).

commit daab37c06dca3f4038223a1a66404eb5cba9a577
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu May 28 16:37:01 2020 +0200

    manager: add duration on events (#37352)