Development #47606
Affiner les messages d'erreur sur échec d'inclusion d'un agenda dans un agenda virtuel
0%
Description
- type de rdv sur agenda 1 = label : 30 minutes, duration : 30, slug = standard
- type de rdv sur agenda 2 = label : 30 min, duration : 30, slug = standard
La différence de label empêche de les avoir ensemble dans un agenda virtuel.
C'est un peu dommage :- ça fait un message d'erreur pas facile à comprendre au premier coup
- reciproquement, quand un agenda est déjà lié à un agenda virtuel, il devient impossible de modifier le libellé des types de rdv, il faut d'abord retirer tous les agendas, tout modifier, tout relier.
J'ai l'impression cependant que techniquement, on pourrait relâcher cette contrainte. J'imagine qu'on ne peut pas aller jusqu'à relacher aussi le slug, et pourtant ça serait sympa aussi... Après tout, seule la durée compte vraiment.
Si on ne peut pas relacher la contrainte, il faudrait clarifier le message d'erreur et rappeler que les label, durée et slug doivent être tous identiques.
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
C'est une décision consciente au moment de l'implémentation, je vais attendre le retour d'Emmanuel là-dessus. (pour référence, ma question sur le sujet dans #37123#note-16 puis la suite).
Mais dès maintenant, le message d'erreur, que disait-il que devrait-il dire pour rendre le problème plus clair ?
Mis à jour par Thomas Noël il y a plus de 3 ans
Le message actuel est « This agenda does not have the same meeting types provided by the virtual agenda. » C'est pas faux mais difficile de voir la différence au premier coup d'oeil (l'erreur était entre "Standard" et "standard"). Peut-être qu'on pourra préciser que l'égalité doit être stricte y compris au niveau de la casse...
Mis à jour par Frédéric Péters il y a plus de 3 ans
Ah de la description du ticket je voyais la différence entre "30 min" et "30 minutes".
Mis à jour par Frédéric Péters il y a plus de 3 ans
cf aussi #37123#note-21, j'écris :
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 :
(...)
(lire la suite là-bas, et peut-être développer davantage que ce qui a été fait)
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
Techniquement la contrainte sur les labels pourrait probablement être complètement relâchée, en théorie celle sur les slugs aussi puisque effectivement seule la durée compte mais c'est plus compliqué (les slugs sont exposés dans les api_fillslot_url et on s'en sert pour retrouver le type de rdv adéquat).
Mais bon je ne sais pas si la situation sera plus compréhensible en relâchant la contrainte sur les labels, qui est là surtout pour être dans une situation certes contraignante mais compréhensible : "il faut que que les agendas aient exactement les mêmes types de rendez-vous, à tous point de vue".
Bref je serai plutôt pour affiner les messages d'erreurs.
Mis à jour par Thomas Noël il y a plus de 3 ans
Emmanuel Cazenave a écrit :
Techniquement la contrainte sur les labels pourrait probablement être complètement relâchée, en théorie celle sur les slugs aussi puisque effectivement seule la durée compte mais c'est plus compliqué (les slugs sont exposés dans les api_fillslot_url et on s'en sert pour retrouver le type de rdv adéquat).
Mais bon je ne sais pas si la situation sera plus compréhensible en relâchant la contrainte sur les labels, qui est là surtout pour être dans une situation certes contraignante mais compréhensible : "il faut que que les agendas aient exactement les mêmes types de rendez-vous, à tous point de vue".
Ca me va. En plus de ce message plus explicite, en cas d'incohérence on devrait afficher la/les différences entre les types présents et ceux de l'agenda qui cherche à être ajouté.
Mis à jour par Frédéric Péters il y a plus de 3 ans
En plus de ce message plus explicite, en cas d'incohérence on devrait afficher la/les différences entre les types présents et ceux de l'agenda qui cherche à être ajouté.
/me sifflote et copie/colle la suite de #37123#note-21 :
<!> 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".
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Sujet changé de permettre des label différents sur les type de rendez-vous des agendas d'un agenda virtuel à Affiner les messages d'erreur sur échec d'inclusion d'un agenda dans un agenda virtuel