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.

Sign up for GitHub

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

Jump to bottom

PASS IAE: utilisation du nouvel encart de PASS dans les formulaires de suspension & prolongation [GEN-2121] #4856

Merged
merged 2 commits into from
Oct 3, 2024
Merged

PASS IAE: utilisation du nouvel encart de PASS dans les formulaires de suspension & prolongation [GEN-2121] #4856

merged 2 commits into from
Oct 3, 2024

Conversation

xavfernandez
Copy link
Contributor

🤔 Pourquoi ?

Pour être cohérent.

🍰 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

@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié labels Sep 30, 2024
@xavfernandez xavfernandez self-assigned this Sep 30, 2024
@notion-workspace Notion Workspace
Copy link

🌿 4/5 [Refonte PASS] Encart du PASS dans les parcours de prolongation / suspension

@xavfernandez xavfernandez changed the title PASS IAE: utilisation du nouvel encart de PASS dans les formulaires de suspension [GEN-2121] PASS IAE: utilisation du nouvel encart de PASS dans les formulaires de suspension & prolongation [GEN-2121] Sep 30, 2024
@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Sep 30, 2024
@github-actions GitHub Actions
Copy link

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

@xavfernandez xavfernandez force-pushed the xfernandez/suspension_approval_box branch from 27814f1 to 62147b7 Compare October 1, 2024 13:11
tonial
tonial approved these changes Oct 3, 2024
View reviewed changes
tests/www/approvals_views/__snapshots__/test_suspend.ambr
@@ -745,6 +822,83 @@
LIMIT 1
''',
}),
dict({
Copy link
Contributor

Choose a reason for hiding this comment

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

vivement qu'on supprime les suspensions 🙄

Il n'y a pas un prefetch que tu peux ajouter pour éviter 3 requêtes de plus ?
Je vois que is_suspended() est fait pour itérer en python sur suspension_set s'il est présent dans le cache
et remainder() utilise self.suspension_set.all() donc les 3 devraient a minima être fait dans une seule requête.
Pour la 3eme requête, ça donne un peu envie d'avoir un approval.ongoing_suspension qui utilise le même cache s'il est présent, mais vu qu'on va supprimer les suspension dans un avenir proche, c'est peut être inutile (enfin, sauf s'il faut gérer l'historique des suspensions pendant longtemps).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

J'ai rajouté un prefetch_related qui permet d'économiser 1 requête sur les 3 et je vais faire une autre PR pour l'approval.ongoing_suspension qui sera utile pour tous les affichages de l'encart 👍

@xavfernandez xavfernandez force-pushed the xfernandez/suspension_approval_box branch from 62147b7 to de3d07a Compare October 3, 2024 14:13
@xavfernandez xavfernandez added this pull request to the merge queue Oct 3, 2024
Merged via the queue into master with commit c10735e Oct 3, 2024
11 checks passed
@xavfernandez xavfernandez deleted the xfernandez/suspension_approval_box branch October 3, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@francoisfreitag francoisfreitag francoisfreitag approved these changes

@tonial tonial tonial approved these changes

@rsebille rsebille Awaiting requested review from rsebille

Assignees

@xavfernandez xavfernandez

Labels
1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xavfernandez @francoisfreitag @tonial