Project

General

Profile

Development #86412

Development #75387: qrcode: ticket chapeau

qrcode: mise en cache du lecteur

Added by Benjamin Dauvergne 8 months ago. Updated 3 months ago.

Status:
Nouveau
Priority:
Normal
Target version:
-
Start date:
01 February 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Discussion initiale: https://pad.entrouvert.org/qrcode#PWA--mise-en-cache-du-lecteur

On va éviter de passer par un service-worker et essayer de se dépatouiller uniquement via l'entête cache-control et les directives max-age et stale-while-revalidate, au niveau des fichiers statiques (#86409) et ici au niveau de la page servie. Le but étant qu'un lien vers un lecteur mis en bookmark (ou reçu par SMS ou email) s'ouvre immédiatement dans le téléphone même hors ligne.

History

#2

Updated by Corentin Séchet 7 months ago

Tu me diras Benj, mais avec https://dev.entrouvert.org/issues/86092 et l'ajout du service worker, la mise en cache du lecteur c'est quelque lignes de JS. Pas d'avis sur ce qui est le mieux techniquement cependant.

#3

Updated by Benjamin Dauvergne 7 months ago

  • Assignee changed from Benjamin Dauvergne to Corentin Séchet

Je pensais qu'on n'utiliserait pas de service worker mais donc si on en a un de toute façon, ok pour le cache via service worker. T'as gagné un ticket.

#4

Updated by Corentin Séchet 7 months ago

  • Status changed from Nouveau to Information nécessaire
  • Assignee changed from Corentin Séchet to Benjamin Dauvergne

Bon, je me suis avancé en disant que ça serait simple : je galère de ouf, j'ai passé une journée là-dessus : https://git.entrouvert.org/entrouvert/passerelle/src/branch/wip/86412-cache-lecteur

Je n'arrive pas à comprendre pourquoi mais je ne cache que le HTML, les requêtes pour les fichiers css & js ne semblent jamais passer par le service worker.

Donc dis moi : soit j'insiste, soit on part sur ta méthode.

#5

Updated by Corentin Séchet 3 months ago

  • Status changed from Information nécessaire to Nouveau
  • Assignee changed from Benjamin Dauvergne to Corentin Séchet

Also available in: Atom PDF