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

Suggestion d'amélioration: Pouvoir pré-remplir les champs à l'aide d'une query string #13

Open
OlivierGrenoble opened this issue Oct 30, 2020 · 15 comments · May be fixed by #62 or #50
Open

Suggestion d'amélioration: Pouvoir pré-remplir les champs à l'aide d'une query string #13

OlivierGrenoble opened this issue Oct 30, 2020 · 15 comments · May be fixed by #62 or #50

Comments

@OlivierGrenoble
Copy link

OlivierGrenoble commented Oct 30, 2020

Bonjour,
Serait il possible de modifier le code pour qu'on puisse pré-remplir les champs à l'aide d'une query string?

Comme ca on pourrait préparer une URL de ce genre:
https://media.interieur.gouv.fr/deplacement-covid-19?field-firstname=Toto&field-lastname=Titi
la mémoriser sur notre téléphone et pouvoir ainsi générer plus facilement nos attestations.

NB: Je n'ai mis que 2 champs dans cet exemple mais on pourrait tous les pré-remplir.

Merci

@pixeye33
Copy link

Bonjour,

Surtout PAS : une query string est envoyée au serveur, cela gâche donc tout le principe de l'application qui est de fonctionner 100% en local (mais ça j'ai aucun doute que les mainteneurs le savent).

Par contre, utiliser des "URI fragment" est bien plus envisageable (tout ce qui est après le # n'est pas envoyé, et est accessible en javascript à la manière de https://www.unkm.fr/)

@pierregoutagny
Copy link

Si l'objectif est de pouvoir mémoriser sur son téléphone, alors cette proposition rejoint #4, et je vous renvoie à la réponse que j'y ai fait, en particulier concernant LAB-MI/attestation-couvre-feu-covid-19#4

@OlivierGrenoble
Copy link
Author

@pierregoutagny J'ai utilisé cette méthode lors du 1er confinement mais elle était moyennement satisfaisante (certains champs ne sont pas mémorisés).

@pixeye33 Dommage pour la solution avec une Query String qui me semblait élégante (je ne conteste pas ce que vous dites sur la requête serveur). Je ne vois pas comment faire la même chose avec des "URI fragment".

@a2br
Copy link

a2br commented Oct 30, 2020

Je m'apprétais à dire exactement la même chose que @OlivierGrenoble
Par rapport aux URI fragments, n'ayant jamais travaillé avec je suppose que ça se récupère sous forme de string, qui peut ensuite être nettoyée (ex: #nom:Doe,prenom:John => {nom:"Doe", prenom:"John"}) et utilisée

@pierregoutagny
Copy link

@OlivierGrenoble avec la PR (du repo sur l'attestation couvre-feu) que j'ai citée, tous les champs étaient retenus (allez voir la démo proposée par l'auteur de la PR).
Cependant, je suis d'accord que la possibilité de charger les informations via l'URL avec les URI fragments, voire le PDF précédent (cf #31), serait bien plus flexible et pratique.

@OlivierGrenoble
Copy link
Author

OlivierGrenoble commented Oct 30, 2020

@pierregoutagny J'ai regardé ce qu'il y a sur le lien auquel tu faisais référence (LAB-MI/attestation-couvre-feu-covid-19#4). Si ma compréhension est bonne, il faut ce changement pour que tous les champs soient mémorisés. Est ce d'ores et déjà utilisable? J'ai l'impression qu'il faudrait que ce changement soit mergé, ce qui n'est pas encore le cas.

@MacKentoch
Copy link

100% d'accord avec @pierregoutagny : inutile de surcharger / sur-complexifier.

Franchement écrire quelque champs dans un formulaire une fois par jour

@tar-gezed
Copy link

Je pensais faire une feature qui stock dans le localStorage les infos a sauvegarder et qui set les date et heure de sortie par défaut à l'heure actuelle, je sais pas si c'est pertinent ?

@viincenb
Copy link
Contributor

Je pensais faire une feature qui stock dans le localStorage les infos a sauvegarder et qui set les date et heure de sortie par défaut à l'heure actuelle, je sais pas si c'est pertinent ?

Je suis d'accord, et c'est déjà en cours: #43 et #44

@jaythegeek
Copy link

Je pense que localStorage c'est le meilleur option, comme ca le user peu effacer par des cookies le data qui est sur leur appareil. Le form avec tous les input pourrais etre mis la dedans. (desolé pour le francais :/)

@a2br a2br linked a pull request Oct 30, 2020 that will close this issue
@betalabmi betalabmi added this to Stockage des données (trop sensible) in Catégorisation des issues et PR Oct 30, 2020
@OlivierGrenoble
Copy link
Author

Savez vous quand il y aura une solution fonctionnelle permettant de mémoriser les champs qu'on a rentré?

christophefontaine added a commit to christophefontaine/attestation-deplacement-derogatoire-q4-2020 that referenced this issue Oct 30, 2020
Fix LAB-MI#13

Les champs disponibles sont:
lastname,firstname,birthday,placeofbirth,address,city,zipcode,reason
- "reason" peut etre repete plusieurs fois pour cocher plusieurs motifs.

- "autogenpdf" simule un click pour une generation automatique du pdf.

exemple:
http://.../#lastname=Dupont&firstname=Jean&birthday=01/01/1950&placeofbirth=Paris&address=zzz&city=Paris&zipcode=75000&reason=travail&reason=achats&autogenpdf
christophefontaine added a commit to christophefontaine/attestation-deplacement-derogatoire-q4-2020 that referenced this issue Oct 30, 2020
Fix LAB-MI#13

Les champs disponibles sont:
lastname,firstname,birthday,placeofbirth,address,city,zipcode,reason
- "reason" peut etre repete plusieurs fois pour cocher plusieurs motifs.

- "autogenpdf" simule un click pour une generation automatique du pdf.

exemple:
http://.../#lastname=Dupont&firstname=Jean&birthday=01/01/1950&placeofbirth=Paris&address=zzz&city=Paris&zipcode=75000&reason=travail&reason=achats&autogenpdf
christophefontaine added a commit to christophefontaine/attestation-deplacement-derogatoire-q4-2020 that referenced this issue Oct 30, 2020
Fix LAB-MI#13

Les champs disponibles sont:
lastname,firstname,birthday,placeofbirth,address,city,zipcode,reason
- "reason" peut etre repete plusieurs fois pour cocher plusieurs motifs.

- "autogenpdf" simule un click pour une generation automatique du pdf.

exemple:
http://.../#lastname=Dupont&firstname=Jean&birthday=01/01/1950&placeofbirth=Paris&address=zzz&city=Paris&zipcode=75000&reason=travail&reason=achats&autogenpdf
@christophefontaine christophefontaine linked a pull request Oct 30, 2020 that will close this issue
@sosnut
Copy link

sosnut commented Oct 30, 2020

C'est tout à votre honneur en tant que développeurs de chercher, et trouver, des moyens de simplifier la tâche de l'utilisateur.
Il faut cependant tenir compte du besoin initial : faire chuter les contacts avec les personnes extérieures, pour sauver des vies.

C'est un avis personnel, mais je pense que ce besoin incite à laisser un peu de friction sous la forme de caractères à retaper, afin d'éviter de faciliter la tâche aux personnes qui génèrent une attestation numérique toutes les heures pour passer la journée dehors et voir tous leurs amis (pas une vue de l'esprit, malheureusement).

Une telle application a un impact en terme de vies humaines, je crois qu'une réflexion est nécessaire avant chaque mise à disposition de fonctionnalités.

@mistraloz
Copy link

mistraloz commented Oct 30, 2020

@sosnut : trouver des moyens simple de compliquer la vie à tout le monde, ce sont les valeurs de la France. Soyons patriote et pourquoi pas dans ce cas ajouter un random pour que les champs se mélangent aléatoirement à chaque affichage ? Voir ajouter un jeu de piste avec les cases à cocher façon guichet numérique (allez cocher la case là-bas si quand vous avez coché la case ci-dessous elle était bleue, ou sinon indiquez le motif en toute lettre dans le courrier postal que nous vous adresserons généralement sous trois semaines).

Tant pis si on doit verbaliser les personnes qui auront mal remplis le formulaire car son ergonomie était respectueuse de nos valeurs nationales.

@a2br
Copy link

a2br commented Oct 31, 2020

Haha, je n'irais pas jusqu'à parler de valeurs nationales, mais plutôt de bon sens. Si quelque chose sert à X, il serait logique de rendre l'accomplissement de X meilleur, plus rapide, plus simple, etc... En tous cas, merci quand même d'avoir partagé votre avis, c'est toujours intéressant d'entendre des gens penser différement, ça ne me serait pas venu à l'esprit. Aussi, merci beaucoup @mistraloz, tu devrais faire une PR pour intégrer ces merveilleuses idées 😂

@pierregoutagny
Copy link

@sosnut quid des personnes en situation de handicap pour qui remplir un formulaire peut prendre encore plus de temps, mais qui ont quand même besoin de se rendre fréquemment chez un professionnel de santé, ou des gens qui s'occupent de leurs parents et qui pourraient avoir à sortir en urgence (pour motifs familiaux impérieux) ?
De toute façon, si ce générateur n'implémente pas d'autofill, des forks avec autofill seront (ou sont déjà) publiés. Je préfère que le générateur que les gens préfèrent utiliser soit celui-ci, plutôt que de savoir qu'il existe une version plus pratique et plus utilisée, mais pas nécessairement à jour ou fiable.

christophefontaine added a commit to christophefontaine/attestation-deplacement-derogatoire-q4-2020 that referenced this issue Nov 28, 2020
Fix LAB-MI#13

Les champs disponibles sont:
lastname,firstname,birthday,placeofbirth,address,city,zipcode,reason
- "reason" peut etre repete plusieurs fois pour cocher plusieurs motifs.

- "autogenpdf" simule un click pour une generation automatique du pdf.

exemple:
http://.../#lastname=Dupont&firstname=Jean&birthday=01/01/1950&placeofbirth=Paris&address=zzz&city=Paris&zipcode=75000&reason=travail&reason=achats&autogenpdf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Catégorisation des issues et PR
Stockage des données (trop sensible)
10 participants