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

Ouvrir le formulaire de contrôle depuis l'onglet "Contrôles" de la fiche navire pour un contrôle existant #2276

Closed
louptheron opened this issue Jul 3, 2023 · 2 comments · Fixed by #2307 or #2631
Assignees
Labels
dev feat. enhancement feature enhancement

Comments

@louptheron
Copy link
Collaborator

louptheron commented Jul 3, 2023

  • Pour voir en détail les informations du contrôles ou de la mission
  • Pour modifier le contrôle depuis la fiche navire

cf. écran : https://xd.adobe.com/view/43ab72bf-f891-43f4-b51c-ca4704cb13ce-16f8/screen/8930f31f-f73b-4664-8338-b08a480e361e/specs/

@louptheron louptheron added feat. enhancement feature enhancement dev labels Jul 3, 2023
@louptheron louptheron changed the title Ouvrir le formulaire de contrôle depuis l'onglet "Contrôles" de la fiche navire Ouvrir le formulaire de contrôle depuis l'onglet "Contrôles" de la fiche navire pour un contrôle existant Jul 3, 2023
@louptheron louptheron self-assigned this Jul 10, 2023
louptheron added a commit that referenced this issue Jul 12, 2023
## Linked issues

- Resolve #2276 
- Affiche un message d'erreur et le retry au lieu du chargement infini
lorsque la requête pour chercher les missions ne trouve pas de mission :
![Screenshot from 2023-07-10
16-18-25](https://github.com/MTES-MCT/monitorfish/assets/14853556/fcc89648-25e4-4fdb-9a32-8034d7246f6f)

----

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

Cela fonctionne bien, après je me pose la question de la justesse du wording qu'on avait fait à l'époque sur le bouton. Est-ce qu'il ne faudrait pas plutôt dire "ouvrir la mission du contrôle" ou bien "ouvrir le contrôle" ? Puisqu'à priori ils vont parfois juste le consulter, sans le modifier ?
@louptheron qu'en penses-tu ?

@louptheron
Copy link
Collaborator Author

On renomme en "Ouvrir le contrôle"

@louptheron louptheron reopened this Oct 17, 2023
louptheron added a commit that referenced this issue Oct 24, 2023
## Linked issues

- Resolve #2342
- Resolve #2276

----

- [ ] Tests E2E (Cypress)
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
Archived in project
2 participants