search

INL / GTB-html

GTB zonder Flash
0 stars 0 forks source link

Cache issue bij update #63

Closed kdepuydt closed 6 years ago

kdepuydt commented 6 years ago

Zorgen dat JS en CSS files allemaal met ?version=DATE of iets dergelijks geladen worden, zodat die files niet uit de cache gehaald kunnen worden.

jan-niestadt commented 6 years ago

Liefst zouden we willen dat het buildscript ervoor zorgt dat iedere keer dat je bouwt er een unieke (bijv. timestamp-based) value wordt gebruikt voor het laden van JS/CSS, om alle cache-issues uit te sluiten.

pmasereeuw commented 6 years ago

Met enig enthousiasme heb ik dit uitgevoerd - wel grondig testen, want er kan zo heel wat omvallen!

jan-niestadt commented 6 years ago

Volgens mij werkt het. Het staat op de (interne) ATO, dus wordt vanzelf nog uitgebreid doorgetest voor we de update op productie uitvoeren.

Overigens was het dus ook genoeg geweest om een "nep-parameter" ?uuid=UUID achter de filenaam te plakken; de files zelf hoeven dus niet per se van naam te veranderen, als de URL maar uniek is. Maar zoals gezegd, het werkt prima zo. Bedankt!

jan-niestadt commented 6 years ago

Wacht, ik sprak te vroeg. Als ik (op de ATO) zoek op bijv. modern lemma "koe*" in alle woordenboeken en dan op de resultaatpagina de paginering probeer te gebruiken, krijg ik de melding "Selecteer a.u.b. tenminste 1 woordenboek".

jan-niestadt commented 6 years ago

Er staan foutmeldingen in de console: image

pmasereeuw commented 6 years ago

Ik vermoed dat het niets te maken heeft met de cache-aanpassing, het probleem treedt op vanaf de commit met hash 07288bc918746193433329a9aedfc454b9f4ca17, waarbij de foutmelding "geen wdb geselecteerd" werd geïntroduceerd.

De foutmelding treedt op als je op lemma "koe" klikt en daarna op "Laatste" (al pas alles op 1 pagina). Het probleem is er niet als je zoekt op "koe" in woord-in-betekenis, vermoedelijk omdat daar wel meer pagina's zijn.

Overigens vind ik ?uuid=UUID mooier - ik was alleen bang dat slimme browsers de querystring bij statische files als *.css niet zouden honoreren. Maar bij nader inzien: hoe moet de browser weten dat dit een statische file is? Dus ik ga daar later toe over (maar het lost het cacheprobleem niet op).

pmasereeuw commented 6 years ago

BTW: moet "tenminste" niet zijn: "ten minste"? Komt vaker voor.

pmasereeuw commented 6 years ago

Fixed, inclusief tenminste

jan-niestadt commented 6 years ago

We hebben die techniek met ?uuid=UUID al vaker gebruikt, en dat ging altijd goed volgens mij. Bedankt voor de fix, staat op de ATO en werkt.