Project

General

Profile

Development #86858

Development #75387: qrcode: ticket chapeau

qrcode: Sauvegarder le timestamp de dernière mise à jour

Added by Corentin Séchet 7 months ago. Updated about 2 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
12 February 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Pour l'instant, tous les évènnements sont téléchargés à chaque démarrage du service worker. Mais le local storage n'est pas accessible dans le service worker : ça demande soit de le sauvegarder côté client (possiblement en lien avec https://dev.entrouvert.org/issues/86857) et de le passer au ServiceWorker lors de l'envoi du message "refreshTally", soit de bricoler quelque chose avec une indexedDB pour y stocker un seul entier.

Associated revisions

Revision 25f0dd1c (diff)
Added by Corentin Séchet about 2 months ago

qrcode: save last update timestamp in service worker (#86858)

History

#1

Updated by Benjamin Dauvergne 7 months ago

Tu pourrais aussi sauvegarder le dernier "since" envoyé par le lecteur dans l'objet Reader (mais pas directement le timestamp, on a aucune garantie qu'il ait été reçu par le lecteur), si since est 0 au lieu de now() - 1 day, tu prends le max de now() - 1 day et reader.last_since ?

#2

Updated by Corentin Séchet 7 months ago

Ca ne fonctionnera pas si on ouvre un lecteur dans deux navigateurs différents, ou qu'on supprime les données de navigation, mais je n'ai pas bien compris la partie à propos de now() - 1 day.

#3

Updated by Benjamin Dauvergne 4 months ago

Ouaip donc sauver dans indexeddb; si tu ne veux pas instantier une nouvelle table tu peux aussi stocker ça dans la même table que les pointages avec une valeur fixe pour "certificate" comme "last-timestamp", ça ne perturbera pas le fonctionnement du reste du code je pense.

#4

Updated by Robot Gitea about 2 months ago

  • Status changed from Nouveau to En cours

Corentin Sechet (csechet) a lié une pull request sur Gitea concernant cette demande :

#5

Updated by Corentin Séchet about 2 months ago

  • Status changed from En cours to Solution proposée
#6

Updated by Benjamin Dauvergne about 2 months ago

  • Status changed from Solution proposée to Solution validée
#7

Updated by Robot Gitea about 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)

Corentin Sechet (csechet) a mergé une pull request sur Gitea concernant cette demande :

#8

Updated by Transition automatique about 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF