Fix/ description observation and manage nested json "type" array #393
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Reviewed-by: andriac
Use same site's logic to display details in table Reviewed-by: andriacap
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bonjour , tout d'abord bravo pour tous ces dev et la sortie de la version 1 de Citizen !
Ensuite dans le cadre d'une prestation pour l'ARB IDF , il été remonté l'absence de "Description" pour les observations ainsi qu'un problème à l'affichage des type "array" récupéré dans le jsonchema.
Ce deuxième problème est également mentionné dans cette issue : #254
Les devs proposés permettent de gérer les champs de type "array" à l'affichage dans le tableau ainsi que de pouvoir afficher la description d'une observation qui pour le moment ne semble pas être fonctionnel car la boucle sur la clé du layout s'arrête au niveau du champ "items". Cette PR résout ce problème .
Concernant l'UX de l'affichage du champ de type "array" , j'ai opté sur l'affichage en mode dépliement du détail de la liste de données lié au champ de type "array" au "click" sur la ligne concernée (voir image ci dessous). Une fois dépliée le background de ligne se met en light-blue. Ce choix se justifie sur le fait de ne pas avoir un tableau à rallonge suivant le nombre d'élément contenu dans le champ de type "array".
Merci d'avance pour votre lecture et je suis biensur à l'écoute de retours / suggestions d'améliorations .
Bonne journée !
Image concernant la réprésentation coté frontend sur la gestion du champ de type "array"
Image concernant l'affichage de la description d'une observation