Skip to content

Admin: Amélioration de l'affichage des critères administratifs [GEN-2131]#5396

Merged
tonial merged 3 commits into
masterfrom
alaurent/boost_admin
Jan 17, 2025
Merged

Admin: Amélioration de l'affichage des critères administratifs [GEN-2131]#5396
tonial merged 3 commits into
masterfrom
alaurent/boost_admin

Conversation

@tonial
Copy link
Copy Markdown
Contributor

@tonial tonial commented Jan 15, 2025 •
edited
Loading

🤔 Pourquoi ?

cf
Les cartes notion
https://www.notion.so/plateforme-inclusion/Am-lioration-de-l-admin-pour-la-certification-des-crit-res-112e8fa5c35b80b48b40c071e6dd03fd?pvs=4
https://www.notion.so/plateforme-inclusion/affichage-null-pour-r-sultat-inconnu-10ee8fa5c35b8007b6d5fa0fd0b074ce?d=112e8fa5c35b80c48768001caf98885b
https://www.notion.so/plateforme-inclusion/est-on-oblig-d-afficher-tous-les-crit-res-s-lectionn-sur-le-Diag-ou-pouvons-nous-restreindre-ceu-112e8fa5c35b800c9434e21576d4681f
et la discussion slack : https://gip-inclusion.slack.com/archives/CU8ATF54L/p1736977461876259

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog. labels Jan 15, 2025
@tonial tonial requested review from EwenKorr and celine-m-s January 15, 2025 21:04
@tonial tonial self-assigned this Jan 15, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 15, 2025

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@tonial tonial force-pushed the alaurent/boost_admin branch from 259aef9 to 013de81 Compare January 15, 2025 21:31
@tonial tonial changed the title Admin: Amélioration de l'affichage des critères administratifs Admin: Amélioration de l'affichage des critères administratifs [GEN-2131] Jan 16, 2025
@tonial tonial force-pushed the alaurent/boost_admin branch 2 times, most recently from 54624ef to 25d5e3a Compare January 16, 2025 21:43
EwenKorr
EwenKorr reviewed Jan 17, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@EwenKorr EwenKorr left a comment

Choose a reason for hiding this comment

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

Beaucoup de nouveautés pour moi, je ne peux pas me risquer à approuver cette PR malheureusement :(

Petit détail peu important : on n'a pas de verbose_name dans SelectedAdministrativeCriteria.administrative_criteria. Maintenant que ça apparaît dans le tableau, on peut peut-être y remédier.

@tonial tonial requested a review from xavfernandez January 17, 2025 10:30
@tonial tonial force-pushed the alaurent/boost_admin branch from 25d5e3a to 1cc2ea2 Compare January 17, 2025 10:31
@tonial
Copy link
Copy Markdown
Contributor Author

tonial commented Jan 17, 2025

Je suis en train d'essayer de comprendre pourquoi les snapshots ont autant bougé dans tests/www/apply/__snapshots__/test_submit.ambr

@xavfernandez
Copy link
Copy Markdown
Contributor

xavfernandez commented Jan 17, 2025

Je suis en train d'essayer de comprendre pourquoi les snapshots ont autant bougé dans tests/www/apply/__snapshots__/test_submit.ambr

A priori, on a les changements sur master donc il faudrait faire un nettoyage.

@tonial tonial force-pushed the alaurent/boost_admin branch from 1cc2ea2 to c16cf78 Compare January 17, 2025 10:47
xavfernandez
xavfernandez approved these changes Jan 17, 2025
View reviewed changes
Comment thread itou/eligibility/admin.py Outdated
Comment on lines +47 to +51
return {
True: mark_safe('<img src="/static/admin/img/icon-yes.svg" alt="True">'),
False: mark_safe('<img src="/static/admin/img/icon-no.svg" alt="True">'),
None: mark_safe('<img src="/static/admin/img/icon-unknown.svg" alt="True">'),
}[obj.certified]
Copy link
Copy Markdown
Contributor

@xavfernandez xavfernandez Jan 17, 2025

Choose a reason for hiding this comment

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

Tu pourrais utiliser from django.contrib.admin.templatetags.admin_list import _boolean_icon

Copy link
Copy Markdown
Contributor Author

@tonial tonial Jan 17, 2025

Choose a reason for hiding this comment

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

🤩

@tonial
eligibility: Improve criterias inline
dfa5f91 
Make fields read-only
Prevent from deleting certified criterias
@tonial tonial force-pushed the alaurent/boost_admin branch from c16cf78 to dfa5f91 Compare January 17, 2025 13:38
@tonial tonial enabled auto-merge January 17, 2025 13:38
@tonial tonial added this pull request to the merge queue Jan 17, 2025
Merged via the queue into master with commit 88deb2f Jan 17, 2025
@tonial tonial deleted the alaurent/boost_admin branch January 17, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EwenKorr EwenKorr EwenKorr left review comments

@xavfernandez xavfernandez xavfernandez approved these changes

@celine-m-s celine-m-s Awaiting requested review from celine-m-s

Assignees

@tonial tonial

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tonial @xavfernandez @EwenKorr