Produits Entr'ouvert » Chrono

Si on trouve le latin plus clair que le grec ça peu être un "Super Agenda", Super Publik aime ça. C'est en tout cas une super fonctionnalité.

Pierre Cros a écrit :

Si on trouve le latin plus clair que le grec ça peu être un "Super Agenda", Super Publik aime ça. C'est en tout cas une super fonctionnalité.

Je propose "agenda virtuel", qui marque bien le fait qu'aucun évènement n'y est directement attaché, que c'est une vue sur un/des agendas normaux, etc.

Mince j'avais raté la proposition de Manu et j'ai dit meta-agenda au club. Mais Agenda virtuel je trouve ça bien. Je peux faire un mail sur la liste du club pour expliquer (si on retient ce nom).

Commentaire général d'une première lecture de l'ensemble des commits,

+def compute_effective_timeperiods(effective_timeperiods, excluded_timeperiods):

J'y aurais plutôt vu une boucle qu'une récursion, mais ok.

+    def accept_meetings(self):
+        if self.kind == 'virtual':
+            return not self.real_agendas.filter(kind='events').exists()

Pourquoi pas un self.real_agendas.filter(kind='meetings').exists(), sans la négation, qui me semblerait ne pas faire prendre de risque aux futures évolutions qui ajouteraient d'autres types d'agenda ?

+        On a virtual agenda we expose transient meeting types based on on the
+        the real ones shared by every real agendas.

Ça me semble la décision importante ici; comment tu arrives à ça plutôt que prendre la totalité des types de rendez-vous ?

+                # select the agenda a desk on the agenda with min fill_rate on the given date

(retirer le premier "the agenda") mais aussi déjà créer un ticket pour prévoir une préférence de remplissage (répartir en mettant là où il y a le moins de rendez-vous, comme ici, ou à l'inverse remplir au maximum un guichet avant d'«ouvrir» le suivant).

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

+  On a virtual agenda we expose transient meeting types based on on the
+        the real ones shared by every real agendas.

Ça me semble la décision importante ici; comment tu arrives à ça plutôt que prendre la totalité des types de rendez-vous ?

Oui c'est clairement le plus important et ce dont on doit parler.

Sur les appels de listing de type de rendez-vous {{agendas_url}}api/agenda/IDENTIFIANT-AGENDA/meetings/, il me semble obligatoire d'agréger pour que ça puisse conserver un sens lorsque c'est présenté dans un formulaire.

Sur les appels de disponibilité des créneaux {{agendas_url}}api/agenda/RENDEZ-VOUS-PASSEPORT/meetings/TYPE-DE-RDV/datetimes/, le type de TYPE-DE-RDV il représente aussi plusieurs MeetingType réels.

Et agréger uniquement sur la base du slug par exemple, ça m'a semblé trop léger, que ça ouvrait trop la porte au mélange patate/carotte involontaire, donc j'ai pris l'option même slug, même durée, même label. Sur le cas d'usage du Nord et des rendez-vous de travailleurs sociaux, c'est raccord.

Coté implémentation sur les appels de disponibilité des créneaux {{agendas_url}}api/agenda/RENDEZ-VOUS-PASSEPORT/meetings/TYPE-DE-RDV/datetimes/, ça me semblait simplifier de savoir qu'on a calcule des créneaux qui ont la même durée sur tous les agendas réels. A y regarder une deuxième fois c'est peut-être pas être pas vraiment fondé.

Aussi ça semblait intéressant de pouvoir compter sur cette hypothèse dans l'UI : quand tu inclues un agenda dans un agendas virtuels, je peux vérifier qu'il a le/les meeting types qui collent avec ce qui est déjà exposé par l'agenda virtuel, pour prévenir des situations du types j'ajoute un nouvel agenda dans un agenda virtuel et deux mois plus tard je me rends compte qu'aucun créneau n'est réservé, ben oui c'est parce qu'il a pas le même slug que les autres sur son/ses meeting type.

Je crée un agenda virtuel, je fais "include agenda", ça me dit "This agenda does not have the same meetingtypes provided by the virtual agenda."; il doit manquer un truc pour autoriser le premier agenda sans le comparer à quelque chose n'existant pas.

Oups désolé, j'avais pas refait un tour dans l'UI après les derniers changements, trop confiant dans mes mes tests qui ne couvraient pas cette situation.

C'est couvert maintenant et le fix en lui même consiste à donner la possibilité à un agenda virtuel d'exposer les types de rendez-vous qu'il aurait si un agenda X n'était pas inclu (la validation est faite dans le clean de VirtualMember, alors que le nouvel agenda est déjà ajouté; ça me sert aussi sur #40057) :

-    def iter_meetingtypes(self):
+    def iter_meetingtypes(self, excluded_agenda=None):
         """ Expose agenda's meetingtypes.
         straighforward on a real agenda
         On a virtual agenda we expose transient meeting types based on on the
         the real ones shared by every real agendas.
         """ 
         if self.kind == 'virtual':
+            base_qs = MeetingType.objects.filter(agenda__virtual_agendas__in=[self])
+            real_agendas = self.real_agendas
+            if excluded_agenda:
+                base_qs = base_qs.exclude(agenda=excluded_agenda)
+                real_agendas = real_agendas.exclude(pk=excluded_agenda.pk)
             queryset = (
-                MeetingType.objects.filter(agenda__virtual_agendas__in=[self])
-                .values('slug', 'duration', 'label')
+                base_qs.values('slug', 'duration', 'label')
                 .annotate(total=Count('*'))
-                .filter(total=self.real_agendas.count())
+                .filter(total=real_agendas.count())
             )
.
.
-        virtual_meetingtypes = self.virtual_agenda.iter_meetingtypes()
+        virtual_meetingtypes = self.virtual_agenda.iter_meetingtypes(excluded_agenda=self.real_agenda)
-
-
-        if len(virtual_meetingtypes) != MeetingType.objects.filter(agenda=self.real_agenda).count():
+
+        num_virt_meetingtypes = len(virtual_meetingtypes)
+        if num_virt_meetingtypes and num_virt_meetingtypes != MeetingType.objects.filter(agenda=self.real_agenda).count():

Toutes les branches sont à jour.

Dans le paramétrage d'un agenda virtuel, la liste des agendas inclus sont repris mais affichent [identiant: whatever] plutôt que le nom de l'agenda. (à mon avis le nom est suffisant, pas besoin d'inclure le slug ici).

_~~

This agenda does not have the same meetingtypes provided by the virtual agenda.

"meeting types" espace.

Je pense que ce message va amener une certaine frustration quand il va falloir aller à la chasse à la différence pour comprendre pourquoi il s'affiche alors que tout a l'air identique; à réfléchir davantage mais peut-être avoir :

  <!> This agenda does not have the same meeting types provided by the virtual agenda.

  * Meeting type "foo" (30 minutes), no meeting with that label.
  * Meeting type "bar" (60 minutes), meeting "bar" existing but with a different duration.
  * Extra meeting type, "baz".

_~~

Deux mini-commentaires pour respirer,

+        # TODO VIRTUAL

Sa suppression a été oubliée dans le commit qui ajoute la prise en charge de l'export/import.

+                self.maximal_booking_delay = 56

Pour suivre Thomas et Benjamin dans #40735, peut-être écrire 8 * 7 plutôt. (yep ça remplace un code où était écrit 56 avant)

_~~

Je réfléchissais aussi à l'affichage d'un agenda virtuel, en me disant qu'il y aurait à l'étendre avec quelques phrases de texte, mais que ça pourrait venir plus tard. Mais du coup j'avais les yeux sur le template :

    {% for real_agenda in agenda.real_agendas.all %}
    <li><a href="{% url 'chrono-manager-agenda-view' pk=real_agenda.pk %}">
        {{real_agenda.label}}
        <span class="identifier">[{% trans "identifier:" %} {{real_agenda.slug}}]</span>
        </a>
    {% endfor %}

et je vois qu'il ne prend pas en compte les paramètres d'accès, qu'on peut du coup avoir un lien vers une erreur si jamais un des agendas n'est pas accessible en lecture.

J'ai tenu compte de tout.

Sur les messages d'erreur je n'arrive à quelque chose d'aussi précis, mais avec tout de même en messages additionnels des :

Meeting type "Foo" (30 minutes) (identifier: foo) does no exist.
Extra meeting type, "Bar".'

J'ai tout poussé, j'ai du faire une modification de dernière minute pour les trads parce que

./agendas/models.py:307 : AVERTISSEMENT : La chaîne de format 'msgid' avec des arguments non nommés ne peut pas être correctement localisée :
                                          Le traducteur ne peut pas réordonner les arguments.
                                          Envisagez d'utiliser une chaîne de format contenant des arguments nommés,
                                          et un mapping plutôt qu'un tuple pour les arguments.

Autre commit fait pour faire ça, signalé par pylint :

-                    slug=virt_meetingtype.slug, label=virt_meetingtype.label, duration=meetingtype.duration
+                    slug=virt_meetingtype.slug, label=virt_meetingtype.label, duration=virt_meetingtype.duration

Le coverage passait bien par là mais ça récupérait l'objet meetingtype d'une boucle précédente.