LAB-MI / attestation-couvre-feu-covid-19

Service de génération de l'attestation de déplacement dérogatoire à présenter dans le cadre du couvre-feu lié au virus covid-19
MIT License
60 stars 15 forks source link

Feat/store and autocomplete data #4

Open theblackhole opened 4 years ago

theblackhole commented 4 years ago

Ajout d'une système permettant d'enregistrer et restaurer automatiquement les données du formulaire

But : Gain de temps, surtout quand la raison est la même tous les soirs (exemple : sortir un animal de compagnie). Après avoir utilisé le formulaire une première fois, il n'y a plus qu'à appuyer sur "Générer l'attestation" pour les suivantes.

Demo : https://autocovid19.oa.r.appspot.com/

Fonctionnalités introduites par cette PR :

Divers/Dev :

Bien sûr, selon vos besoins je peux adapter cette PR (par exemple en n'enregistrant pas certains champs comme la raison, ou retirer l'autocompletion avec l'heure actuelle pour ne garder que la date, ou encore retirer les changements de l'.editorconfig). Mais personnellement, pour des raisons pratiques (j'ai un chien à sortir tous les soirs, donc toujours la même raison à chaque fois), je préfère utiliser ma propre version et compte la proposer à mon entourage dès qu'elle sera publiée.

Idées d'évolution dans un second temps avec maj UI/UX :

theblackhole commented 4 years ago

Demo : https://autocovid19.oa.r.appspot.com/

theblackhole commented 4 years ago

(Petit rebase pour résoudre les conflits de la PR #6)

theblackhole commented 4 years ago

En vérifiant ma PR après ce rebase, je viens de voir que -conformément aux instructions de l'editorconfig- les fichiers sur lequels j'ai travaillé ont eu leurs sauts de ligne convertis au format Unix (LF) au lieu de Windows (CRLF) précédemment. C'est une bonne pratique pour une collaboration cross-plateform d'avoir des sauts de lignes LF mais par contre ça rend les changements difficiles à voir sur github car toutes les lignes sont modifiées (CRLF > LF).

Souhaitez-vous que je convertisse à nouveau en CRLF pour faire ressortir uniquement les changements apportés par cette feature ?

theblackhole commented 4 years ago

Pour info l'instance de demo, https://autocovid19.oa.r.appspot.com/ , pointe dorénavant sur la nouvelle attestation de confinement (version adaptée par @tar-gezed sur attestation-deplacement-derogatoire-q4-2020#58)

Pour retrouver l'ancienne demo qui pointe sur l'ancienne attestation couvre-feu, c'est à cette adresse : https://1-0-0-rc3-dot-autocovid19.oa.r.appspot.com/

Je garde cette issue ouverte dans le cas où la situation sanitaire s'améliorerait et qu'on reviendrait vers un système de couvre-feu au lieu de confinement total :) (sauf si un nouveau repo est encore créé à cette occasion hahaha)