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

Bug d'enregistrement de fichiers sur la page de suivi usager #1074

Closed
emilschn opened this issue Mar 21, 2023 · 3 comments
Closed

Bug d'enregistrement de fichiers sur la page de suivi usager #1074

emilschn opened this issue Mar 21, 2023 · 3 comments
Assignees
Labels
bug Quelque chose ne fonctionne pas v1.7.4
Milestone

Comments

@emilschn
Copy link
Collaborator

emilschn commented Mar 21, 2023

Après analyse, il semblerait qu'au moment d'avoir ajouté un suivi avec des images, un utilisateur se soit impatienté, ait cliqué plusieurs fois, et ait créé 4 suivis à la suite.
C'est un "bug" mais il n'est pas bloquant. Reproduit une fois pour l'instant.

Possibilité d'UI pour éviter cela : lorsque l'utilisateur clique sur "Ajouter" pour valider le formulaire, on passe le bouton en désactivé, le temps que le formulaire se valide.


Erreur Sentry liée :
https://sentry.incubateur.net/organizations/betagouv/issues/29742/?project=61&query=is%3Aunresolved+is%3Afor_review+assigned_or_suggested%3A%5Bme%2C+none%5D&sort=inbox&statsPeriod=14d

@emilschn emilschn created this issue from a note in Backlog 🪵 (🟢 Dans le sprint - To do) Mar 21, 2023
@emilschn emilschn self-assigned this Mar 22, 2023
@emilschn emilschn moved this from 🟢 Dans le sprint - To do to 💻 Sprint - Doing in Backlog 🪵 Mar 22, 2023
@emilschn
Copy link
Collaborator Author

D'après les analyses Sentry, on a 4 fois l'événement.
4 fois à la même heure.
On a 4 fois le suivi associé.
D'après moi, la personne a été impatiente, et a cliqué 4 fois sur "valider", donc ça a copié une fois les images, mais après, elles n'y étaient plus. Et donc ce n'était plus possible de les copier.

Plusieurs façons de voir d'après moi :

  • c'est un cas spécifique, on attend de voir si ça se reproduit, de toute façon, ça ne bloque pas, on ne fait rien
  • on peut ajouter un test sur la copie du fichier pour vérifier si il est toujours présent
  • on peut ajouter de l'UI pour que, quand on valide le formulaire, le bouton se mette en attente, pour éviter de re-valider le formulaire bêtement

Quoiqu'il en soit, je l'enlève du TODO, je le mets "A trier" en spécifiant un peu le contexte.

Qu'en pensez-vous ? @hmeneuvrier @sfinx13

@emilschn emilschn moved this from 💻 Sprint - Doing to 🟠 Priorité moyenne in Backlog 🪵 Mar 22, 2023
@hmeneuvrier
Copy link
Collaborator

Ok, pas d'urgence ! Test sur la copie du fichier ça me parait bien, éventuellement UI

@hmeneuvrier hmeneuvrier added the bug Quelque chose ne fonctionne pas label Mar 28, 2023
@hmeneuvrier hmeneuvrier moved this from 🟠 Priorité moyenne to 🟢 Dans le sprint - To do in Backlog 🪵 Mar 28, 2023
@emilschn emilschn moved this from 🟢 Dans le sprint - To do to 💻 Sprint - Doing in Backlog 🪵 Mar 31, 2023
@emilschn emilschn moved this from 💻 Sprint - Doing to ⚠️ Sprint - Code revue in Backlog 🪵 Mar 31, 2023
@sfinx13 sfinx13 moved this from ⚠️ Sprint - Code revue to 🔍 Sprint - Tests in Backlog 🪵 Apr 3, 2023
@hmeneuvrier hmeneuvrier moved this from 🔍 Sprint - Tests to ✅ Done in Backlog 🪵 Apr 4, 2023
@sfinx13 sfinx13 added the v1.7.4 label Apr 4, 2023
@sfinx13
Copy link
Collaborator

sfinx13 commented Apr 4, 2023

Faudra suveiller sentry @hmeneuvrier @emilschn

@sfinx13 sfinx13 closed this as completed Apr 4, 2023
@mathildepoulpux mathildepoulpux moved this from ✅ Done to Archives - Anciens sprint in Backlog 🪵 Dec 7, 2023
@mathildepoulpux mathildepoulpux added this to the v1.7.4 milestone Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Quelque chose ne fonctionne pas v1.7.4
Projects
None yet
Development

No branches or pull requests

4 participants