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

GPS: Ajout de liens dans l'admin #5383

Merged
merged 1 commit into from
Jan 14, 2025
Merged

GPS: Ajout de liens dans l'admin #5383

merged 1 commit into from
Jan 14, 2025

Conversation

tonial
Copy link
Contributor

@tonial tonial commented Jan 14, 2025

🤔 Pourquoi ?

https://www.notion.so/gip-inclusion/Faire-figurer-les-groupes-de-suivi-sur-les-pages-utilisateurs-de-l-admin-1735f321b6048037828cdf72eb037f24?pvs=4

🍰 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 self-assigned this Jan 14, 2025
Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

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

Est-ce que ces lookup_allowed ne viennent pas en doublon de follow_up_groups_or_members ?
Si on veut les infos, cela semble plus simple d'aller sur la fiche de l'utilisateur ?

itou/users/admin.py Outdated Show resolved Hide resolved
@tonial
Copy link
Contributor Author

tonial commented Jan 14, 2025

Le allow_lookup c'est pour éviter une 404 : Filtering by memberships__member not allowed

@tonial tonial force-pushed the alaurent/gps_admin branch 2 times, most recently from 10bd518 to 1274c39 Compare January 14, 2025 08:11
@tonial tonial requested a review from xavfernandez January 14, 2025 08:30
Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

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

Ah oui effectivement, tu ne fais qu'un lien vers la vue liste, je pensais avoir vu un Inline.
Ça mériterait peut-être des petits tests ?

@tonial
Copy link
Contributor Author

tonial commented Jan 14, 2025

Je me suis dit que vu que c'était une feature pour GPS c'était pas grave 🙈
Je vais ajouter un test rapido

@tonial tonial force-pushed the alaurent/gps_admin branch from 1274c39 to a02014d Compare January 14, 2025 08:53
@tonial tonial enabled auto-merge January 14, 2025 08:53
@tonial tonial added this pull request to the merge queue Jan 14, 2025
Merged via the queue into master with commit 9e8c24e Jan 14, 2025
9 checks passed
@tonial tonial deleted the alaurent/gps_admin branch January 14, 2025 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants