[Missions] Affichage d'une unité déjà mobilisé #878
Conversation
louptheron
commented
Oct 12, 2023
•
louptheron
commented
- Resolve [Missions] Afficher un message d'avertissement quand une mission est déjà ouverte sur une unité monitorfish#2480
frontend/src/features/missions/MissionForm/ControlUnitsForm.tsx
Outdated
Show resolved
Hide resolved
| @@ -13,7 +13,7 @@ import { ReactComponent as DeleteSVG } from '../../../uiMonitor/icons/Delete.svg | |||
|
|
|||
| import type { ControlUnit } from '../../../domain/entities/controlUnit' | |||
|
|
|||
| export function ControlUnitSelector({ controlUnitIndex, controlUnitPath, removeControlUnit, ...props }) { | |||
| export function ControlUnitSelector({ controlUnitIndex, controlUnitPath, isEngaged, removeControlUnit, ...props }) { | |||
There was a problem hiding this comment.
pourquoi ne pas mettre le hook ici ? tu passes juste l'id en props et fait le useMemo avec le find direct dans le ueMemo.
T'en penses quoi?.
There was a problem hiding this comment.
Discuté sur Mattermost : Je ne cache pas avec RTK les requêtes car je pense qu'il faut avoir cette information tout le temps à jour, du coup si je mets le hook dans le sous-composant, il va y a voir potentiellement plusieurs appels APIs.
There was a problem hiding this comment.
Après discussion avec Thomas, il y a un cache par défaut de 60 secondes dans RTK, ce cache ne s'invalide pas tant qu'il y a un subscriber (lorsqu'il n'y a plus de subscriber, le timeout de 60 secondes est enclenché pour invalider le cache).
Ce fonctionnement n'est pas optimal pour le CACEM car ils laissent ouvert le formulaire des missions toute la journée.
Nous allons donc utiliser un pollingInterval et descendre le hook dans le sous-composant.
louptheron
force-pushed
the
loup/add-engaged-control-units-warning
branch
from
940ae2c
to
1f5b6dd
Compare
