Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Affichage des infos utiles dans l'admin habilitation #576

Merged
merged 1 commit into from
Mar 14, 2022

Conversation

tut-tuuut
Copy link
Member

🌮 Objectif

Améliorer l'interface d'administration :

  • Afficher au moins les infos "respo structure" et "dpo" sur une demande. Ces infos étaient masquées dans l'état actuel des choses.
  • Permettre de voir facilement quelles demandes ont été posées par un demandeur.

🔍 Implémentation

  • Utilisation de formulaires Inline pour éviter d'avoir à rajouter des pages de liste pour les DPO et les Respo Structure, avec tous les risques que ça supposait sur l'intégrité des données.
  • Utilisation du module django-reverse-admin pour pouvoir utiliser des formulaires Inline sur des champs OneToOneField. Ça donne une drôle de tête au code, mais ça rend exactement comme je veux dans l'admin, donc je ne me plains pas.

🖼️ Images

Dans l'accueil de l'admin, on ne liste toujours que les demandes et les demandeurs, rien d'autre :

Capture d’écran 2022-03-11 à 16 53 22

Page d'un demandeur, qui liste ses demandes :

Screenshot 2022-03-11 at 16-53-04 Agnès Haasser Modification de Demandeur Site d’administration de Django

Page d'une demande, qui contient plein d'infos :

Screenshot 2022-03-11 at 16-52-29 Modification de Demande d’habilitation Site d’administration de Django

@tut-tuuut
Copy link
Member Author

)
readonly_fields = ("issuer_id",)
inlines = (OrganisationRequestInline,)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Franchement, pouvoir faire des interfaces aussi complètes et fonctionnelles juste en déclarant des classes et des attributs… c'est ❤️

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Django ❤️

Comment on lines +60 to +64
inline_type = "stacked"
inline_reverse = (
"manager",
"data_privacy_officer",
)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Là c'est le mode de déclaration des Inline pour le module reverse-admin. Ça m'embête que ça n'ait pas la même jolie tête que les inlines standards, mais ça marche (j'ai testé en lecture et en écriture). Donc tant pis.

@@ -10,6 +10,7 @@ django-referrer-policy==1.0
django-tabbed-admin==1.0.4
django-import-export==2.7.1
django-debug-toolbar==3.2.4
django-reverse-admin==2.9.6
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tiens je connaissais pas cette librairie !

@tut-tuuut tut-tuuut merged commit 9e511f5 into main Mar 14, 2022
@tut-tuuut tut-tuuut deleted the ahr/display-useful-info-in-habilitation-admin branch March 14, 2022 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants