Produits Entr'ouvert » Chrono

Nécessité à cette occasion d'étendre Event pour marquer l'Event "primaire", pour permettre à l'annulation de libérer le lot.

Nécessité à cette occasion d'étendre Event pour marquer l'Event "primaire", pour permettre à l'annulation de libérer le lot.

Il fallait bien sûr lire "Booking" et pas "Event" ici; et j'ai finalement fait le taf dans #15218.

Sur le principe ok mais

• le endpoint doit aussi gérer les agendas de type événement,
• ça se lit quand même comme un gros besoin de factoriser les choses, je trouve (le fillslot devant devenir un appel à fillslots(events=[event]))

        slots = request.data.get('slots')
        if not slots:
            return Response({'err': 1, 'reason': 'missing slots'})
        if not isinstance(slots, list):
            return Response({'err': 1, 'reason': 'slots must be a list'})

Idéalement j'aimerais qu'on arrête ça et qu'on puisse avoir django-rest-framework assurer cette partie.

On n'est quand même pas trop loin de pouvoir gérer la réservation des slots dans le même guichet, il me semble qu'il suffirait de available_desks = set(Desk.objects.filter(id=available_desk_id)) et prendre l'intersection et à la fin voir s'il reste quelque chose. (et si c'est aussi simple, autant l'avoir dès maintenant).

Frédéric Péters a écrit :

Sur le principe ok mais

• le endpoint doit aussi gérer les agendas de type événement,

Un poil de mal à voir l'usage, mais ok.

• ça se lit quand même comme un gros besoin de factoriser les choses, je trouve (le fillslot devant devenir un appel à fillslots(events=[event]))

Mouais, j'avais commencé et ça faisait un peu le bronx partout, mais ok pour revoir donc la gestion de fillslot comme un cas particulier.

Idéalement j'aimerais qu'on arrête ça et qu'on puisse avoir django-rest-framework assurer cette partie.

#22979 à relire d'abord, donc

On n'est quand même pas trop loin de pouvoir gérer la réservation des slots dans le même guichet, il me semble qu'il suffirait de available_desks = set(Desk.objects.filter(id=available_desk_id)) et prendre l'intersection et à la fin voir s'il reste quelque chose. (et si c'est aussi simple, autant l'avoir dès maintenant).

J'ai pas bien pigé, mon patch gère bien la résa de x slots sur le même guichet (enfin disons qu'il "gérait", je le refais completely)

J'ai pas bien pigé, mon patch gère bien la résa de x slots sur le même guichet (enfin disons qu'il "gérait", je le refais completely)

Ok j'ai du mal lire cette partie, peut-être je me suis fait avoir available_desk = Desk.objects.filter(id=available_desk_id)[0] qui m'a fait penser qu'on piochait dans une liste de guichet (alors que non, id=available_desk_id ça donnera jamais qu'un unique guichet (et donc plutôt .get(...), je trouve).

ok pour moi.