Development #46655
/manage/: exposer le slug des collectivités
0%
Description
Et en permettre l'édition.
Via #22250#note-12,
Ce problème là plaide simplement pour exposer le champ slug (dans l'IHM), le slug est sensé être immuable mais pas de manière absolue, ce n'est pas un UUID, on doit pouvoir corriger un erreur temporaire de nommage à la main.
Files
Associated revisions
History
Updated by Nicolas Roche 4 months ago
- File 0001-manager-allow-ou-slug-edition-46655.patch 0001-manager-allow-ou-slug-edition-46655.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Puisque #22250 calcule le slug quand il est absent,
j'ai fait en sorte qu'il n’apparaisse pas sur la page d'ajout.
Updated by Benjamin Dauvergne 4 months ago
- Status changed from Solution proposée to En cours
J'aurai préféré comme ailleur et dans l'admin Django le bout de JS qui fait le calcul du slug à la volée, ça me parait justifier que des fois on définisse le slug à la main, mais si c'est trop de taf et bien on pourra toujours le faire juste après.
Bon par contre tu pourrais l'afficher sur la page de détail et dans le listing aussi.
Updated by Frédéric Péters 4 months ago
J'aurai préféré comme ailleur et dans l'admin Django le bout de JS qui fait le calcul du slug à la volée,
Communément (w.c.s., chrono, combo) on ne fait pas ça, on propose uniquement à l'édition, après coup, de revenir dessus. (dans passerelle oui on propose dès la création du connecteur mais non il n'y a pas de saisie auto à la volée javascript).
Updated by Benjamin Dauvergne 4 months ago
Frédéric Péters a écrit :
Communément (w.c.s., chrono, combo) on ne fait pas ça, on propose uniquement à l'édition, après coup, de revenir dessus. (dans passerelle oui on propose dès la création du connecteur mais non il n'y a pas de saisie auto à la volée javascript).
J'ai cru le voir quelque-part qui n'était pas une admin Django, j'ai peut-être rêvé, mais soit si un usage s'est établi oublions. Reste juste d'afficher ce slug dans le listing et les détails pour qu'on puisse avoir envie de le modifier si ce n'est pas ce qu'on espérait.
Updated by Nicolas Roche 4 months ago
J'aurai préféré comme ailleur et dans l'admin Django le bout de JS qui fait le calcul du slug à la volée,
C'est fait uniquement sur les pages d'ajout des connecteur :
https://passerelle.dev.publik.love/manage/actesweb/add
(pas sur les pages des requêtes par exemple)
https://passerelle.dev.publik.love/manage/opendatasoft/test/query/new/
En fait j'ai fait en sorte de cacher le slug sur la page d'ajout.
Ce n'est pas souhaité ?
Reste juste d'afficher ce slug dans le listing et les détails
J'affiche déjà le slug dans la page des détails :
https://authentic.dev.publik.love/manage/organizational-units/1/
Tu souhaites vraiment faire apparaître cette info technique sur le listing ?
(actuellement il n'apparaît que le libellé et si c'est, ou non, la collectivité par défaut.)
https://authentic.dev.publik.love/manage/organizational-units/
Updated by Benjamin Dauvergne 4 months ago
Nicolas Roche a écrit :
J'aurai préféré comme ailleur et dans l'admin Django le bout de JS qui fait le calcul du slug à la volée,
C'est fait uniquement sur les pages d'ajout des connecteur :
https://passerelle.dev.publik.love/manage/actesweb/add
(pas sur les pages des requêtes par exemple)
https://passerelle.dev.publik.love/manage/opendatasoft/test/query/new/En fait j'ai fait en sorte de cacher le slug sur la page d'ajout.
Ce n'est pas souhaité ?
Ok, on verra a l'usage.
Reste juste d'afficher ce slug dans le listing et les détails
J'affiche déjà le slug dans la page des détails :
https://authentic.dev.publik.love/manage/organizational-units/1/
Tu souhaites vraiment faire apparaître cette info technique sur le listing ?
(actuellement il n'apparaît que le libellé et si c'est, ou non, la collectivité par défaut.)
https://authentic.dev.publik.love/manage/organizational-units/
Oui la page est plutôt vide autant l'afficher, c'est une information utile en général (déjà pour voir qu'on a un nommage un peu cohérent des slugs entre eux).
Updated by Nicolas Roche 4 months ago
- File 0001-manager-allow-ou-slug-edition-46655.patch 0001-manager-allow-ou-slug-edition-46655.patch added
- Status changed from En cours to Solution proposée
J'ai juste modifié OUTable (et le test) :
fields = ('name', 'slug', 'default')
Updated by Benjamin Dauvergne 4 months ago
- Status changed from Solution proposée to Solution validée
Updated by Nicolas Roche 4 months ago
- Status changed from Solution validée to Résolu (à déployer)
commit 03f2b80a98ef1d38530647e214fdffedcd70d90b
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Tue Sep 22 12:35:31 2020 +0200
manager: allow ou slug edition (#46655)
Updated by Frédéric Péters 4 months ago
- Status changed from Résolu (à déployer) to Solution déployée
manager: allow ou slug edition (#46655)