Produits Entr'ouvert » Chrono

0001: # incomplete rules make some events not bookable
Peut-être qu'il faudrait prévoir de pouvoir afficher dans la démarche que le calendrier de garde n'est pas complet, pour avoir un warning ?
Donc, dans un autre ticket, avoir un endpoint pour récupérer les détails d'un agenda de garde ?

0003:

            if max_start:
                all_periods.filter(date_start__lte=max_start)
            if min_start:
                all_periods.filter(date_end__gte=min_start)

custody_exceptions = all_periods.exclude(guardian__user_external_id=guardian_external_id) J'ai pas l'impression que ça soit utilisé dans le code ?

Lauréline Guerin a écrit :

0001: # incomplete rules make some events not bookable
Peut-être qu'il faudrait prévoir de pouvoir afficher dans la démarche que le calendrier de garde n'est pas complet, pour avoir un warning ?
Donc, dans un autre ticket, avoir un endpoint pour récupérer les détails d'un agenda de garde ?

Ouep, c'est pour un autre ticket et c'est à voir avec Stéphane également.

0003:
[...]
gte ou gt ? La borne max d'une période doit être exclue
(peut-être modifier ou compléter les tests pour avoir une occurence le jour de la date de fin d'une SharedCustodyPeriod, pour vérifier ?)

C'est purement de l'optimisation, on pourrait récupérer toutes les périodes ça fonctionnerait pareil, ce n'est pas là qu'on vérifie la date pour de vrai. Mais oui il manque un test ça ne doit pas être couvert actuellement

custody_exceptions = all_periods.exclude(guardian__user_external_id=guardian_external_id) J'ai pas l'impression que ça soit utilisé dans le code ?

Si juste en dessous agenda.prefetched_exceptions.extend(custody_exceptions). En fait si il y a une période de garde explicite pour un gardien il faut faire deux choses, l'inclure pour l'un et l'exclure pour l'autre, et ici l'exclusion marche comme s'il y avait une exception donc autant réutiliser ça.

Si juste en dessous agenda.prefetched_exceptions.extend(custody_exceptions). En fait si il y a une période de garde explicite pour un gardien il faut faire deux choses, l'inclure pour l'un et l'exclure pour l'autre, et ici l'exclusion marche comme s'il y avait une exception donc autant réutiliser ça.

Ok vu, je cherchais prefetched_exceptions dans les patchs du ticket, mais c'est dans le code existant.

Mais oui il manque un test ça ne doit pas être couvert actuellement

J'attends de voir le test avant de valider :)

Automatic expiration