Project

General

Profile

Développement #47606

Affiner les messages d'erreur sur échec d'inclusion d'un agenda dans un agenda virtuel

Added by Thomas Noël over 4 years ago. Updated about 1 year ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
13 October 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No
Tags:

Description

Typiquement on a deux agendas qui possèdent ce type de rendez-vous :
  • 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.

History

#1

Updated by Frédéric Péters over 4 years ago

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 ?

#2

Updated by Thomas Noël over 4 years ago

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...

#3

Updated by Frédéric Péters over 4 years ago

Ah de la description du ticket je voyais la différence entre "30 min" et "30 minutes".

#4

Updated by Frédéric Péters over 4 years ago

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)

#5

Updated by Emmanuel Cazenave over 4 years ago

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.

#6

Updated by Thomas Noël over 4 years ago

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é.

#7

Updated by Frédéric Péters over 4 years ago

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".
#8

Updated by Emmanuel Cazenave over 4 years ago

  • Subject changed from permettre des label différents sur les type de rendez-vous des agendas d'un agenda virtuel to Affiner les messages d'erreur sur échec d'inclusion d'un agenda dans un agenda virtuel
#9

Updated by Valentin Deniaud about 1 year ago

  • Tags set to tri2023

Also available in: Atom PDF