[BO - signalement] Créer la modale d'upload de photos/documents #2298

numew · 2024-02-27T15:04:46Z

Ticket

#2200

Description

Création de la modale d'upload des document (https://xd.adobe.com/view/3cae505c-cf52-408c-a800-0bdd78e695f2-69fd/screen/0c3f48bb-09e0-49bd-9251-1d45685f6cf8/) qui s'active via le feature flipping FEATURE_DOCUMENTS_ENABLE
Création de la modale d'upload de photos (https://xd.adobe.com/view/3cae505c-cf52-408c-a800-0bdd78e695f2-69fd/screen/2d6299ec-c7ff-4f4b-ae2e-b311cd6932d0/) qui s'active via le feature flipping FEATURE_DOCUMENTS_ENABLE

Pré-requis

make load-migrations
make npm-build

Tests

Tester l'upload de documents via le bouton "Ajouter des documents" de la fiche signalement avec et sans la feature FEATURE_DOCUMENTS_ENABLE
Tester l'upload de photos via le bouton "Ajouter des photos" de la fiche signalement avec et sans la feature FEATURE_DOCUMENTS_ENABLE

hmeneuvrier · 2024-02-28T16:56:00Z

J'ajoute une photo dans un signalement fait avec l'ancien formulaire, et j'ai quand même la sélect box "sélectionner un désordre"
le bouton "fermer" de la modale ne fonctionne pas
Même si je n'ai rien sélectionné dans la selctbox, je peux valider l'ajout de photo (et le type est Autre, tant mieux)
Je peux glisser des pdf dans la dropzone de la modale "photos", pour cette modale il faudrait vérifier le type mime des fichiers (et après malgré son extension il est quand même considéré comme photo et pas comme document)
dans la selectbox de la modale photos, j'ai tous les désordres. Mais je n'ai pas la possibilité de choisir "Autre" comme expliqué dans le texte de la modale
sur la modale de documents, si je glisse 2 documents dans la dropzone, la selectbox n'apparait que pour 1 des 2 documents
idem pour la modale de photos

assets/styles/histologe.scss

assets/controllers/back_signalement_view/back_signalement_view.js

src/Controller/Back/SignalementFileController.php

sfinx13

Aujourd'hui l'upload multiple génère un suivi donc un envoi d'email il faudrait garder ce fonctionnement.

numew · 2024-02-29T14:18:31Z

@hmeneuvrier pour te répondre point par point :

Ok, c'est corrigé
Ok, c'est corrigé
Le fonctionnement est le suivant : on ajoute les photo (en utilisant la route habituelle, et donc elle passe en Autre par défaut) et ensuite la sélection du désordre correspond a une modification. Donc effectivement on peux valider immédiatement (je n'ai pas mis de bouton annuler pour cette raison)
Ok, c'est corrigé
Étant donnée qu'on enregistre "Autre" par défaut + le fait en lien avec le point 1 que l'on ne propose pas forcément de sélection, c'est plutot le texte de la modale qui est à revoir ?
et 7 Je n'ai pas eu le cas lors de mes tests, est ce n'etait pas un problème temporaire de ton coté ?

sfinx13

A voir avec @mathildepoulpux
J’enlèverai le bouton valider, ça peut laisser penser que les fichiers n'ont pas été envoyé.
Quant à la sauvegarde des désordres elle se ferait instantanément (à la sélection) ça va sans doute demander à ce qu'on envoi un feedback pour dire le désordre est bien sauvegardé

mathildepoulpux · 2024-02-29T16:07:59Z

A voir avec @mathildepoulpux J’enlèverai le bouton valider, ça peut laisser penser que les fichiers n'ont pas été envoyé. Quant à la sauvegarde des désordres elle se ferait instantanément (à la sélection) ça va sans doute demander à ce qu'on envoi un feedback pour dire le désordre est bien sauvegardé

J'ai pas compris de quoi on parle 😁

numew · 2024-03-01T09:03:34Z

Notes faisant suite à la discussion en stand up :
Étant données l'impossibilité d'avoir des progress bar unitaire et un envoie de fichier en masse est décidé ce qui suit

Les uploads restent unitaire mais ne crée pas de suivis
Les fichiers crées sont tagués "à notifier" (champ en BDD)
Une fois tous les fichiers uploadés, appel d'une route qui va générer un suivi regroupant les fichiers tagué "à notifier" du signalement

hmeneuvrier · 2024-03-04T10:12:41Z

@numew @sfinx13

ok pour l'upload unitaire et l'ajout d'un champ "à notifier" pour faire un suivi unique
du coup, pas ok pour enlever le bouton "valider", car je pense que c'est ce bouton valider qui doit valider la sélection du type et envoyer la notification (à discuter aussi avec @mathildepoulpux je l'ajoute au point UX)
est-ce qu'il ne faudrait pas un bouton "supprimer" en face des fichiers avant de valider ?
pour le fait de ne pas sélectionner de désordre et que ça mette "AUTRE" automatiquement, voir avec @mathildepoulpux si ça lui convient de modifier le texte de la modale et que le "AUTRE" soit implcite ?

hmeneuvrier · 2024-03-04T14:16:03Z

Point dev UX :

on dtoi pouvoir annuler dans la modale (garder le bouton, et garder le fonctionnement du bouton valider qui doit faire quelquechose)
rajouter un bouton pour supprimer les fichiers individuellement
enlever la phrase "si vous ne trouvez pas le désordre, choisissez AUTRE"

numew · 2024-03-04T17:19:07Z

En principe tout fonctionne

hmeneuvrier · 2024-03-05T13:59:21Z

J'ajoute une photo de type non accepté (heic), j'ai le message d'erreur, et si je clique sur "annuler", j'ai une erreur :

hmeneuvrier · 2024-03-05T14:00:36Z

Quand je ferme la modale sans Valider après avoir ajouté des photos, les photos sont qwuand même ajoutées au signalement (idem pour les documents)

templates/_partials/_modal_upload_files.html.twig

hmeneuvrier · 2024-03-05T13:57:36Z

assets/controllers/back_signalement_view/back_signalement_view.js

+        }
+        let div = document.createElement('div')
+        div.classList.add('fr-alert', 'fr-alert--error', 'fr-alert--sm')
+        div.innerHTML = `Le type du fichier ${file.name} n'est pas accepté (acceptés : ".jpg", ".png", ".gif")`;


Je pense qu'il faudrait remonmter le message d'erreur, car là il est sous la ligne de flottaison, et on ne le vois pas forcément.

Comme tout dépend de la taille de l'écran j'ai plutôt fait en sorte que le contenu de la modale remonte

je ne vois pas de différence avec avant 🤔

J'ai mis le message au dessus

numew · 2024-03-06T11:34:40Z

Quand je ferme la modale sans Valider après avoir ajouté des photos, les photos sont qwuand même ajoutées au signalement (idem pour les documents)

C'est inversé

hmeneuvrier · 2024-03-06T15:42:46Z

Quand je ferme la modale sans Valider après avoir ajouté des photos, les photos sont qwuand même ajoutées au signalement (idem pour les documents)

OK

hmeneuvrier · 2024-03-06T15:44:55Z

J'ajoute une photo de type non accepté (heic), j'ai le message d'erreur, et si je clique sur "annuler", j'ai une erreur.

OK maintenant

assets/controllers/back_signalement_view/back_signalement_view.js

src/Controller/Back/SignalementFileController.php

sonarcloud · 2024-03-08T11:31:03Z

Quality Gate passed

Issues
18 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

sfinx13

Lecture et test OK

hmeneuvrier

Relecture et test OK

numew requested review from hmeneuvrier, sfinx13 and emilschn February 27, 2024 15:04

sfinx13 requested changes Feb 29, 2024

View reviewed changes

sfinx13 reviewed Feb 29, 2024

View reviewed changes

numew marked this pull request as draft March 1, 2024 08:55

numew marked this pull request as ready for review March 4, 2024 12:45

hmeneuvrier reviewed Mar 5, 2024

View reviewed changes

sfinx13 requested changes Mar 7, 2024

View reviewed changes

assets/controllers/back_signalement_view/back_signalement_view.js Outdated Show resolved Hide resolved

sfinx13 reviewed Mar 7, 2024

View reviewed changes

src/Controller/Back/SignalementFileController.php Outdated Show resolved Hide resolved

numew added 7 commits March 8, 2024 11:56

modal upload documents #2200

54b47d1

change sbased on helene comments #2200

85be881

changes based on saidi comment #2200

8c71a40

add suivi with delay #2200

757f670

call route back_signalement_file_waiting_suivi #2200

8d485b6

option d'annulation #2200

50e4797

change based on comments #2200

7f8cd63

numew added 3 commits March 8, 2024 12:04

remove files on close modal by default

2f5e45a

change position error message #2200

7409b56

change based on saidi comments #2200

fdf821e

numew force-pushed the feature/2200-copy-upload-modal branch from c1e8cc3 to fdf821e Compare March 8, 2024 11:05

corrections impact rebase #2200

47250c0

sfinx13 approved these changes Mar 11, 2024

View reviewed changes

hmeneuvrier approved these changes Mar 11, 2024

View reviewed changes

hmeneuvrier merged commit a08fa03 into develop Mar 11, 2024
3 checks passed

hmeneuvrier deleted the feature/2200-copy-upload-modal branch March 15, 2024 10:48

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BO - signalement] Créer la modale d'upload de photos/documents #2298

[BO - signalement] Créer la modale d'upload de photos/documents #2298

numew commented Feb 27, 2024 •

edited

Loading

hmeneuvrier commented Feb 28, 2024

sfinx13 left a comment •

edited

Loading

numew commented Feb 29, 2024

sfinx13 left a comment •

edited

Loading

mathildepoulpux commented Feb 29, 2024

numew commented Mar 1, 2024

hmeneuvrier commented Mar 4, 2024

hmeneuvrier commented Mar 4, 2024

numew commented Mar 4, 2024

hmeneuvrier commented Mar 5, 2024

hmeneuvrier commented Mar 5, 2024 •

edited

Loading

hmeneuvrier Mar 5, 2024

numew Mar 5, 2024

hmeneuvrier Mar 6, 2024

numew Mar 7, 2024

numew commented Mar 6, 2024

hmeneuvrier commented Mar 6, 2024

hmeneuvrier commented Mar 6, 2024

sonarcloud bot commented Mar 8, 2024

sfinx13 left a comment

hmeneuvrier left a comment

[BO - signalement] Créer la modale d'upload de photos/documents #2298

[BO - signalement] Créer la modale d'upload de photos/documents #2298

Conversation

numew commented Feb 27, 2024 • edited Loading

Ticket

Description

Pré-requis

Tests

hmeneuvrier commented Feb 28, 2024

sfinx13 left a comment • edited Loading

Choose a reason for hiding this comment

numew commented Feb 29, 2024

sfinx13 left a comment • edited Loading

Choose a reason for hiding this comment

mathildepoulpux commented Feb 29, 2024

numew commented Mar 1, 2024

hmeneuvrier commented Mar 4, 2024

hmeneuvrier commented Mar 4, 2024

numew commented Mar 4, 2024

hmeneuvrier commented Mar 5, 2024

hmeneuvrier commented Mar 5, 2024 • edited Loading

hmeneuvrier Mar 5, 2024

Choose a reason for hiding this comment

numew Mar 5, 2024

Choose a reason for hiding this comment

hmeneuvrier Mar 6, 2024

Choose a reason for hiding this comment

numew Mar 7, 2024

Choose a reason for hiding this comment

numew commented Mar 6, 2024

hmeneuvrier commented Mar 6, 2024

hmeneuvrier commented Mar 6, 2024

sonarcloud bot commented Mar 8, 2024

Quality Gate passed

sfinx13 left a comment

Choose a reason for hiding this comment

hmeneuvrier left a comment

Choose a reason for hiding this comment

numew commented Feb 27, 2024 •

edited

Loading

sfinx13 left a comment •

edited

Loading

sfinx13 left a comment •

edited

Loading

hmeneuvrier commented Mar 5, 2024 •

edited

Loading