magopian / att

ATTESTATION DE DÉPLACEMENT DÉROGATOIRE
https://magopian.github.io/att
32 stars 9 forks source link

autofill form #10

Closed xgaia closed 4 years ago

xgaia commented 4 years ago

Hello,

I add some JS to autofill the form:

fix #4

xgaia commented 4 years ago

I change the checkboxes with radio because the attestation concern only one move

certifie que mon déplacement ...

magopian commented 4 years ago

I'm not sure about the url parameters: if there are url parameters, it's going to pass them to the server. And I don't want to have (apache) logs on the server listing names and signatures and adresses :/

Natim commented 4 years ago

Good point, maybe we can generate a URL with a #hash instead? That would be build when entering the form the first time?

Natim commented 4 years ago

base64.urlsafe_encode(json.dumps({"name": "Xavier Garnier", "address": "3 rue du X à Blah", "city": "Blah", "dob": "21/02/1988"}))

magopian commented 4 years ago

but then it would be reversible, what's the point?

Maybe I don't understand the use case here? Is it to have multiple "saved" documents? If so, we can think of something else, like having a url parameter that doesn't have any PII like a uuid, or a neutral name like "attestation-mathieu" and so on. And then the data would use that as the key for the localStorage item.

Natim commented 4 years ago

Je pense que le use case c'est de pouvoir partager l'adresse entre ton tel et ton ordi par exemple ou entre deux ordinateurs si tu le rempli sur un ordinateur et que tu souhaites l'imprimer sur un autre.

xgaia commented 4 years ago

Oui c'est ca. Et aussi quand le cache est supprimé. Bon apres c'est vrai que c'est pas ouf si l'URL est passé au serveur. J'avais pas penser a ca. Je peu revert ca, mais garder la date auto et les radio a la place des checkboxes.

xgaia commented 4 years ago

Même si maintenant une attestation numérique n'est pas valable finalement :'(

Natim commented 4 years ago

Même si maintenant une attestation numérique n'est pas valable finalement :'(

On peut toujours l'utiliser pour imprimer :+1:

magopian commented 4 years ago

On peut toujours l'utiliser pour imprimer

ce n'est même plus sûr, probablement que non : https://twitter.com/Place_Beauvau/status/1240236276416118784

magopian commented 4 years ago

Il reste deux sujets non traités :

Je vais donc maintenant fermer cette PR, et vous remercier pour le travail effectué, les retours et feedback. Merci !