Développement #92260
Clients d'API, améliorer l'interface d'ajout d'adresses IP
0%
Description
Actuellement c'est un textarea qui prend une adresse par ligne, avec possibilité de commentaires en faisant commencer une ligne par un #.
Proposition d'améliorer ça avec un champ de formulaire plus chiadé : deux champ texte simple côte à côte, un pour l'IP un pour le commentaire, et un bouton pour ajouter une nouvelle ligne, cf capture jointe.
Derrière ça demande un changement en base de donnée pour ne plus stocker ça comme du texte, j'imagine qu'un jsonfield qui contiendrait [{'ip': '192.168.0.1', 'comment': 'blabla'}, ...] conviendrait. Pas besoin de migration puisque c'est encore sous feature flag.
J'espère qu'il n'y a pas grand chose à inventer niveau implémentation car c'est ce qui existe déjà dans chrono du côté des champs personnalisés d'un type d'évènement (#63285).
Files
History
Updated by Benjamin Dauvergne (retour le 1er août) about 1 year ago
- Priority changed from Normal to Bas
Mais pourquoi ?
Updated by Valentin Deniaud about 1 year ago
Ça me semble plus ergonomique et homogène avec le reste de Publik. Le code devrait être plus simple, et plus facile à maintenir/faire évoluer (passage par la validation django, stockage structuré).
J'imagine qu'en terme de nombre de lignes ça va en ajouter un peu mais on peut penser à mutualiser un jour dans gadjo, alors on sera également gagnant de ce point de vue.
C'est à améliorer tant que le feature flag est en place, après ce sera relou.
Updated by Benjamin Dauvergne (retour le 1er août) about 1 year ago
Mais c'est déjà ergonomique, tu tapes des IPs et ça marche, tu les ranges comme tu veux, tu mets des sauts de ligne pour faire des groupes, t'est content, tu peux copier coller la liste d'IP qu'on t'a envoyé par mail. Les interfaces texte c'est bien, faut juste pas en abuser.
À la rigueur on pourrait donner un exemple dans l'interface pour montrer que ça gère les commentaires et c'est 100% plus ergonomique que n'importe quelle autre interface stérile avec deux colonnes IPs/Commentaire et où il faut aller lire la doc pour comprendre.
Updated by Valentin Deniaud about 1 year ago
Benjamin Dauvergne a écrit :
Mais c'est déjà ergonomique
Je suis nul en ergonomie mais il me semble qu'une interface qui soit capable de valider la saisie et de pointer précisément où il y a une erreur c'est une plutôt bonne pratique.
Si tu veux davantage objectiver il y a des gens qui s'y connaissent en accessibilité dans les parages, on peut leur demander de trancher.
où il faut aller lire la doc pour comprendre.
Tu exagères, si simplement avoir une colonne « Commentaire » pose question on peut imaginer un autre intitulé ou davantage d'explication via placeholder ou n'importe quoi d'autre.
Updated by Benjamin Dauvergne (retour le 1er août) about 1 year ago
Valentin Deniaud a écrit :
Je suis nul en ergonomie mais il me semble qu'une interface qui soit capable de valider la saisie et de pointer précisément où il y a une erreur c'est une plutôt bonne pratique.
Ok je vais ouvrir un ticket pour avoir une validation, ça me parait suffisant.
Updated by Benjamin Dauvergne (retour le 1er août) about 1 year ago
Benjamin Dauvergne a écrit :
Ok je vais ouvrir un ticket pour avoir une validation, ça me parait suffisant.
Il y a déjà une validation de toutes les IPs et netmasks qui sont dans le champ.