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

Exemptions pour les alertes "Absence de message FAR" et "Pêche par un navire tiers en ZEE française" #1370

Closed
AdelineCelier opened this issue Sep 15, 2022 · 2 comments · Fixed by #2383 or #2568
Assignees
Labels
dev feat. enhancement feature enhancement

Comments

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Sep 15, 2022

Après discussion avec Céline et Xavier G., il en est ressorti que cette logique d'exemption ne concernait que ces deux alertes, et qu'il n'était donc pas nécessaire de l'implémenter dans les paramètres de toutes les alertes.

Pour rester le plus simple possible dans un premier temps, il a été convenu que les deux listes seraient éditées via des tableaux dans le backoffice, un peu sur le même modèle que les segments de flotte. Le SIP pourrait se charger des exemptions JPE, et la reg ou l'ops des vénézueliens autorisés à venir pêcher en eaux françaises.

A noter que pour Céline ce point n'était plus vraiment une priorité, puisque les navires concernés ont vu leurs alertes suspendues, au moins jusqu'au 31/12. Il faudrait donc que les tableaux soient prêts pour début 2023.

A faire : dessiner rapidement les tableaux pour gérer les exemptions

@louptheron
Copy link
Collaborator

louptheron commented Aug 24, 2023

Je rattache l'exemption des navires à l'ajout de l'alerte de FAR manquant de 48h (et la validation automatique des FAR manquants).

Nous avons convenu que ces exemptions seraient des suspensions d'alertes (objet déjà existant dans MonitorFish) ; ça simplifie l'architecture et le développement back + data de garder cette structure générique plutôt que de créer un nouvel objet exemptions.
Les suspensions d'alertes seront donc affichées dans un nouvel onglet des alertes, qui comprend toutes les façades.

louptheron added a commit that referenced this issue Sep 15, 2023
…tion d'alertes (#2383)

## Linked issues

- [x] Resolve #2189
- [x] Resolve #1370
- [x] Permettre la mise sous silence des alertes quand il n'y a pas de
façade (considéré comme "Toute façade")
- [x] Passer sur l'API publique `api/v1`
- [x] Enlever `MISSING_FAR_ALERT` de l'API `operational_alerts`
@louptheron
Copy link
Collaborator

louptheron commented Oct 3, 2023

Bug à corriger

  • Lorsque je rajoute un navire exempté, son flagState est en minuscule alors qu'il devrait être en majuscule, ce qui fait planter l'API.

louptheron added a commit that referenced this issue Oct 3, 2023
## Linked issues

- Resolve #1370 
- Resolve #2556

----

- [ ] Tests E2E (Cypress)
@louptheron louptheron moved this to Done in Board MonitorFish May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment