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

Préavis – Consulter la liste des préavis #2843

Closed
louptheron opened this issue Jan 29, 2024 · 8 comments · Fixed by #2982, #3382 or #3387
Closed

Préavis – Consulter la liste des préavis #2843

louptheron opened this issue Jan 29, 2024 · 8 comments · Fixed by #2982, #3382 or #3387
Assignees
Labels
dev feat. enhancement feature enhancement

Comments

@louptheron
Copy link
Collaborator

louptheron commented Jan 29, 2024

En tant qu'unité, au bureau, je veux pouvoir consulter la liste des préavis pour :

  • Identifier le meilleur port et le meilleur moment pour la sortie
  • Identifier un ou plusieurs navires à contrôler

image

@ivangabriele
Copy link
Member

ivangabriele commented Jan 31, 2024

  • Traduction FR des segments => A faire via le BackOffice
  • MultiCascader ou CheckTreePicker pour le filtre ports (vs engins) ?
  • Filtres & colonnes
    • Diffusé ou non
    • Navire préciblé ou non
    • On retire "status" pour l'instant.
  • Bouton Télécharger = PDF des préavis cochés
  • Bouton Envoyer = Option à challenger (email à envoyer à des adresses avec les PDFs des préavis)
  • Heure de débarque PNO => vérifier que ça ne puisse pas être nul (1)

image

@ivangabriele ivangabriele changed the title Préavis - Consulter la liste des préavis Préavis – Consulter la liste des préavis Feb 2, 2024
ivangabriele added a commit that referenced this issue Mar 27, 2024
## Linked issues

- Resolve #2484
- Resolve #2843
- Resolve #2844


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **New Features**
- Introduced a comprehensive system for monitoring fishing activities
through prior notifications, including vessel information, catches, and
risk assessments.
- Added functionality to retrieve and filter prior notifications and
logbook reports based on various criteria such as dates, locations, and
vessel details.
- Implemented a new frontend component for displaying a list of prior
notifications with capabilities for filtering, sorting, and accessing
detailed views.

- **Enhancements**
- Enhanced the retrieval of prior notifications with additional details
like port names and seafront information.
- Refined the vessel entity representation in the frontend to improve
data handling and presentation.

- **Bug Fixes**
- Fixed issues related to the incorrect handling of null values in
vessel identity properties.

- **Tests**
- Added unit and integration tests to ensure the reliability of new
functionalities related to prior notifications and logbook reports.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
ivangabriele added a commit that referenced this issue Apr 15, 2024
…es navires ou celle des facteurs de risque [WORKAROUND] (#3108)

## Linked issues

- #2843
- #2867
- #3107

----

- [ ] Tests E2E (Cypress)


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **New Features**
- Introduced a new interface for managing vessel risk factors, enhancing
the ability to retrieve and map risk factors based on internal reference
numbers.
- Enhanced vessel information retrieval capabilities with new functions
and caching mechanisms in repository classes.

- **Bug Fixes**
- Standardized naming conventions across various classes and methods to
ensure consistency in risk factor management.

- **Refactor**
- Renamed multiple repository and class references to align with updated
naming conventions.

- **Documentation**
- Updated test cases and comments to reflect changes in functionality
and improve clarity.

- **Chores**
- Implemented new database entries for vessels and risk factors to
support testing and development.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Apr 16, 2024

Revue de la vue liste des préavis

Vue générale

  • Je me rends compte que je n'avais pas fait l'écran correctement (le voici maintenant), mais il faut rajouter un bouton sur la carte pour accéder directement à la vue préavis de la side window. Et tant qu'on est sur ces boutons raccourcis sur la carte, est-ce qu'il serait possible de :

    • faire correspondre l'ordre des boutons entre le menu de la carte et celui de la side window : 1. couches carto / 2. navires favoris / 3. missions / 4. alertes / 5. préavis / 6. VMS 👍🏻
    • revoir l'espacement entre les icônes sur la carte : 16 px entre les couches cartos et les navires favoris, puis entre les navires favs et le groupe d'icônes suivantes, puis 4 px entre chaque icône du groupe missions/alertes/préavis/VMS
    • mettre la bonne icône pour les couches cartos, et mettre plutôt l'étoile en filaire qu'en plein :
      Map layers Favorite
  • Changer les titres des différentes vues listes pour l'adapter à l'entrée du menu des façades : "Tous les préavis" / "Préavis MEMN", "Préavis NAMO", etc. 👍🏻

  • Il manque les drapeaux des nationalités des navires

  • Il n'y a pas de données dans la colonne "débarque prévue"
    => ces données seront normalement visibles en prod

  • Il y a le même bug sur les menus déroulants que sur le menu déroulant des espèces dans les missions : quand je scrolle dans le menu, je ne peux pas cliquer sur les valeurs. En revanche quand je fais une recherche cela fonctionne
    => ticket à part ouvert sur le sujet, Recherche menu déroulants – Problème de chargement des valeurs au scroll #3211

  • Le menu déroulant "date du dernier contrôle" est un peu trop long (blanc en-dessous) et un peu trop large aussi. 👍🏻
    image

  • Il manque la petite phrase juste en haut à gauche du tableau, qui dit "x préavis (tous les horaires sont en UTC)" 👍🏻

  • Il manque les titles sur les colonnes tronquées 👍🏻

  • Le bouton-lien "réinitialiser les filtres" est un peu bas, le placer à 16 px du dernier élément de filtre (soit menu déroulant, soit tag ou date picker "période spécifique)
    image
    image

  • On avait parlé de mettre le nom des segments de flotte en français, @VincentAntoine est-ce qu'on a ça en base qque part ?
    => ticket créé sur le sujet, le Segments de flotte – Traduire en français les noms des segments dans le backoffice #3212

  • Il faudrait que les éléments déroulés du date range picker (calendrier ou liste d'horaires) se rangent quand on clique ailleurs. Là ils restent affichés tant qu'on n'a pas cliqué à l'intérieur, ce qui n'est pas pratique quand on a saisi les infos au clavier. 👍🏻

Vue détaillée des lignes de préavis

  • Les libellés en Slate Gray des différents champs semblent en medium ou en bold, il faut qu'ils soient en regular.
  • Principales espèces à bord : l'idée est de n'afficher que les 5 premières espèces en termes de tonnage. Si l'utilisateur veut en voir plus, il clique sur "voir plus de détail" et ça ouvre la fiche préavis 👍🏻
  • Rajouter le tag "acquitté" ou "non acquitté", ainsi qu'un tag "message corrigé" au niveau de la colonne PNO émis (cf. cet écran) 👍🏻
  • Quand on déplie la ligne de tableau d'un préavis, il faut que visuellement l'espace qui se déroule soit "intégré" à la ligne, il ne faut pas d'espace entre les deux
    image
  • Quand un préavis n'a pas de segment, est-ce qu'on peut plutôt mettre "pas de segment" plutôt qu'un tiret : ce n'est pas que la donnée manque, c'est que le navire n'appartient à aucun segment.

@ivangabriele
Copy link
Member

ivangabriele commented Apr 23, 2024

Il n'y a pas de données dans la colonne "débarque prévue"

À gérer côté data à priori de ce que j'ai checké.

UPDATE

Vu avec Vincent : ce sera ajouté aux nouveaux PNOs lors de l'enrichissement mais ceux existants sur le staging n'en n'ont pas en effet.

On avait parlé de mettre le nom des segments de flotte en français, @VincentAntoine est-ce qu'on a ça en base qque part ?

Same, à gérer côté data à priori de ce que j'ai checké.

UPDATE

Important

Il faut changer tous les noms en français dans le back-office AVANT qu'on enrichisse les données en prod.

Sur le staging je ne sais pas trop par contre.

Quand on déplie la ligne de tableau d'un préavis, il faut que visuellement l'espace qui se déroule soit "intégré" à la ligne, il ne faut pas d'espace entre les deux

Ça je sens que ça va être un joli challenge UI, les seules solutions que je vois pour l'instant demandent à revoir la structure du composant commun qu'on utilise pour les tableau. Je vais en discuter pour voir si une idée plus simple existe.

@ivangabriele
Copy link
Member

Je rajoute un point :

  • Quand j'ouvre le date picker ("Arrivée estimée du navire entre deux dates"), je ne peux pas le fermer en cliquant autre part dans la fenêtre

@louptheron Normalement c'est fixé sur ma PR préavis en cours ça (tout comme sur les missions d'ailleurs)

ivangabriele added a commit that referenced this issue Apr 23, 2024
## Linked issues

- #2843

----

- [ ] Tests E2E (Cypress)
@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Apr 26, 2024

Revue n°2

=> bien noté pour l'enrichissement des préavis en prod.
=> on peut s'atteler à la traduction des segments avant la mise en prod finale, ce sera fait !

  • Il arrive que l'app freeze quand on consulte les préavis

  • Rajouter un loader quand les filtres chargent 👍🏻

  • Pour les types de préavis : mettre "pas de type de préavis" quand le PNO ne rentre dans un aucun type. Normalement on doit toujours avoir la data. 👍🏻

  • J'ai l'impression que l'icône des préavis (le poisson) est un peu petite : dans les large buttons sur la carte, les icônes doivent faire 26x26 px

  • L'emplacement pour les icônes de drapeaux est bien là, mais elles ne s'affichent pas 👍🏻
    image

  • Pour les types de préavis : il y a qques petites retouches à faire

    • renommer les types "espèces soumises à préavis" en "autres espèces soumises à préavis"
    • prioriser l'ordre d'apparition des types de préavis dans le tableau ? D'abord "préavis tiers/communautaire", puis les préavis "port désigné" (pas de priorité au sein de ceux-ci), puis les préavis "autres espèces soumises à préavis" => Je l'ai fait mais il faut retirer le tri sur cette colonne si on fait ça 👍🏻
  • Sur les segments :

    • quand un navire a plusieurs segments, il faut les séparer par une virgule et un espace, et non pas par un slash 👍🏻
    • dans le détail de la ligne, revoir le nom des segments : reprendre le code des segments, un tiret et le nom en français, et passer à la ligne pour chaque segment 👍🏻
  • Possible d'élargir un peu le bloc de texte du détail des 5 principales espèces à bord, pour limiter le nb de retours à la ligne ? Passer de 206 à 224 px par exemple. 👍🏻

  • Pour le "pas de segment", possible de le mettre en forme comme dans les maquettes ? En Slate gray et en italique, pour qu'il se différencie plus visuellement des segments. 👍🏻

  • De mon côté j'ai toujours l'impression que les libellés des champs en Slate Gray ne sont pas en regular (mais ptet que je vois mal ??) 👍🏻
    image

  • Tag "x signalements" :

    • le texte du tag doit être en Maximum Red 👍🏻
    • gérer le pluriel sur le title 👍🏻
  • Reprendre les espacements tels que définis pour l'écran en 1680 px
    image

  • Concernant le point sur le fait d'intégrer la ligne dépliée à la première ligne du tableau (pas d'espace entre les deux) :
    si c'est un challenge UI, on le dépriorise ce n'est pas archi important

    • Est-ce que c'est possible à minima de retirer le hover sur la ligne dépliée ? Comme elle n'est pas cliquable c'est un peu étrange qu'elle passe en Blue Yonder quand on la survole. 👍🏻
      image

@ivangabriele ivangabriele reopened this May 21, 2024
ivangabriele added a commit that referenced this issue May 28, 2024
## Linked issues

- #2843
- #2867

----

- [x] Tests E2E (Cypress)
ivangabriele added a commit that referenced this issue May 28, 2024
## Linked issues

- #2843
- #2867

----

- [ ] Tests E2E (Cypress)
ivangabriele added a commit that referenced this issue May 29, 2024
ivangabriele added a commit that referenced this issue May 29, 2024
…geur de la fiche (#3236)

## Linked issues

- #2843
- #2867

----

- [ ] Tests E2E (Cypress)
ivangabriele added a commit that referenced this issue May 30, 2024
…é des façade (#3244)

## Linked issues

- #2843

----

- [ ] Tests E2E (Cypress)
@AdelineCelier
Copy link
Collaborator

AdelineCelier commented May 30, 2024

@ivangabriele

  • Retirer le scroll horizontal qui empêche de voir les lignes du tableau en entier 👍
  • renommer les types "espèces soumises à préavis" en "autres espèces soumises à préavis" => @VincentAntoine je crois que c'est une tâche pour toi ! ❌
  • Le bouton-lien "voir plus de détail" n'apparaît que dans le cas où il y a plus de 5 espèces dans le préavis, et qu'elles ne s'affichent donc pas toutes dans le détail de la ligne TODO @louptheron 👍
  • Dans MF pour le CNSP (pas ext), quand on clique sur le bouton-lien "voir plus de détail", cela renvoie vers le formulaire éditable TODO @louptheron
  • Dans MF CNSP, ne mettre que des boutons "éditer" en fin de ligne du tableau, dans MF terrain, ne mettre que des boutons "voir" (œil) en fin de ligne ❌
  • l'icône des préavis (le poisson) sur la carte est un peu petite : dans les large buttons sur la carte, les icônes doivent faire 26x26 px TODO @louptheron 👍
  • Mettre un espace de 8 px entre le bas du tableau et le bouton "charger les 10 préavis suivants" TODO @louptheron 👍
  • Dans le détail de la ligne d'un préavis, pour les principales espèces à bord : si le nom de l'espèce et sa quantité est plus longue que 224 px, possible de le couper avec trois pointillés et mettre un title ? TODO @louptheron 👍
  • Raccourir le bouton en largeur (taille du texte) 👍

louptheron added a commit that referenced this issue Jul 10, 2024
## Linked issues

- Resolve #3242
- Resolve #3240 
- Resolve #2843

----

- [ ] Tests E2E (Cypress)
@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Jul 10, 2024

@VincentAntoine

  • les préavis "espèces soumises à préavis" n'ont pas été renommées en "autres espèces soumises à préavis" 👍🏻

@louptheron

  • quand je clique sur "voir plus de détail", cela charge indéfiniment le formulaire sans l'afficher 👍
  • ce sont encore des icônes "œil" en fin de ligne pour les préavis auto dans MF pour le CNSP 👍
  • l'icône sur la carte est à la bonne taille, en revanche je ne suis pas certaine mais elle ne me semble pas en Gainsboro mais en White...il faut qu'elle soit en Gainsboro quand le bouton n'est pas sélectionné et en White quand il l'est (bon c'est un petit détail je reconnais) 👍

Les 2 points encore non traités sont dans des tickets :

louptheron added a commit that referenced this issue Jul 12, 2024
## Linked issues

- Resolve #3242
- Resolve #3240
- Resolve #3319
- Resolve #2843
- Resolve #3241
- Resolve #3317
- Resolve #2869

----

- [ ] Tests E2E (Cypress)
@AdelineCelier
Copy link
Collaborator

Les derniers points me semblent réglés !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev feat. enhancement feature enhancement
Projects
Status: Done
3 participants