Produits Entr'ouvert » Chrono

Note : dans meta, je propose d'éviter des noms de clé avec des "-" (donc éviter "no-bookable-datetimes") car ça en rendra l'usage peu simple dans les appels du style webservice.foo.meta.key

Ici on pourrait avoir "empty": true s'il n'y a aucune place disponible.

Oui j'ai écrit empty dans l'autre ticket mais en fait je ne suis pas fan parce que trop imprécis, je préfère une clé plus longue, mais tout à fait avec des underscores et pas des tirets.

Je préférerais un seul commit, pris isolément le 0001 amène trop de questions qui sont sans doute répondues quand se trouve en même temps considéré le contenu de 0002.

-        response = {'data': [get_event_detail(request, x, agenda=agenda) for x in entries]}
+        response = {
+            'data': [get_event_detail(request, x, agenda=agenda) for x in entries],
+        }

Ta configuration black ne doit pas correspondre à chrono, ce changement ne devrait pas apparaître.

                'no_bookable_datetimes': bookable_datetimes_number_available == 0,

affaire perso mais sur ce genre je préfère un explicite bool() qui indique l'affaire dès le début de la lecture, i.e.

                'no_bookable_datetimes': bool(bookable_datetimes_number_available == 0),

            # we do not have the := operator, so we do that

Il était là avant et pas bien clair mais il me semble que c'est tout à fait inutile désormais, que le truc d'appeler make_id() c'était pour ne pas recalculer deux fois le slot_id dans la list comprehension, mais que comme elle a été dégagée, il deviendrait bien plus simple et lisible de :

        for slot in generator_of_unique_slots:
            # Make virtual id for a slot, combining meeting_type.id and
            # iso-format of date and time.
            # !!! The datetime must always be in the local timezone and the local
            # timezone must not change if we want the id to be stable.
            # It MUST be a garanty of SharedTimePeriod.get_time_slots() !!!
            slot_id = ...
            data.append(
                    {
                        'id': slot_id,

et à nouveau à parler de commentaire je ne comprends pas la dernière ligne de celui-ci, soit on explicite/complète, soit on dégage.

Est-ce que pour les tests que tu ajoutes, l'utilisation des fixtures mock_now ou utilisant mock_now (virtual_meetings_agenda, some_data) est utile ?
Est-ce qu'on a vraiment besoin de tester les timezones et l'heure courante ?
Ces fixtures multiplient les tests (x9), j'ai tendance à éviter de les utiliser quand on peut s'en passer.