Project

General

Profile

Development #46655

/manage/: exposer le slug des collectivités

Added by Frédéric Péters 4 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
15 Sep 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 03f2b80a (diff)
Added by Nicolas Roche 4 months ago

manager: allow ou slug edition (#46655)

History

#1

Updated by Nicolas Roche 4 months ago

  • Assignee set to Nicolas Roche
#2

Updated by Nicolas Roche 4 months ago

Puisque #22250 calcule le slug quand il est absent,
j'ai fait en sorte qu'il n’apparaisse pas sur la page d'ajout.

#3

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.

#4

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

#5

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.

#6

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/

#7

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

#8

Updated by Nicolas Roche 4 months ago

J'ai juste modifié OUTable (et le test) :

fields = ('name', 'slug', 'default')

#9

Updated by Benjamin Dauvergne 4 months ago

  • Status changed from Solution proposée to Solution validée
#10

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)

#11

Updated by Frédéric Péters 4 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF