feat(front): ignore les orientation manager sans comptes pour l'assig… #1925
Conversation
HadrienMP
force-pushed
the
fix/rattachement-carnet-co-sans-compte
branch
from
5afe2d0
to
458650e
Compare
|
La review app a été déployée : https://cdb-app-review-pr1925.osc-fr1.scalingo.io. |
| <script lang="ts" context="module"> | ||
| export const toOrientationManagerOptions = (result: GetOrientationManagerQuery) => | ||
| result.orientation_manager | ||
| .filter((om) => !!om.account) |
There was a problem hiding this comment.
Est-ce que ça ne fonctionnerait pas aussi d’ajouter le filtre à la requête dans _getOrientationManager.gql ?
There was a problem hiding this comment.
Ça pourrait apporter une amélioration marginale de perf. Mais ça ne change pas le côté nullable dans le type.
On pourrait ignorer le côté nullable en se disant que le filtre de la requête suffit.
There was a problem hiding this comment.
Oui j’avais justement en tête que ça permettrait d’avoir moins de code à maintenir coté client — en faisant effectivement l’impasse sur la nullabilité du type.
J’ai aussi testé de modifier la requête pour faire quelque chose comme account(where:{type:{_eq:orientation_manager}}) { id orientation_manager { … } } ce qui présente l’avantage de garantir la présence d’un account (qui est bien en définitive l’objet qu’on cherche à sélectionner), sachant que là aussi, le système de types ne permet pas d’exprimer le fait que dans ce résultat de requête orientation_manager sera toujours présent.
|
🎉 This PR is included in version 1.246.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
…nation
🔧 Problème
En production, certains orientation manager n'ont pas de compte (account). Cet état de données bloque l'affichage de la modale d'assignation d'un OM à un.e bénéficiaire.
Fix: #1913
🍰 Solution
Ignorer les orientation manager sans comptes
🚨 Points d'attention / Remarques
RAS
🏝️ Comment tester
Les tests automatiques doivent passer