openstreetmap-polska / aed-mapa

Mapa defibrylatorów AED
https://OpenAEDMap.org
MIT License
16 stars 8 forks source link

PWA i czyszczenie cache #51

Closed Ancymonek closed 2 years ago

Ancymonek commented 2 years ago

Występuje u mnie sytuacja, że mam: 'dodaliśmy już 1186 AED', po odświeżeniu jest już 'dodaliśmy już 1300 AED'. Gdy wracam na stronę mam znowu 1186.. Może by ograniczyć cache jedynie do niektórych plików, bądź jakoś czyścić cache .geojsonów?

@ksciana

ttomasz commented 2 years ago

Serwer powinien zwracać w nagłówkach żeby dane (geojson) cacheować przez godzinę, a resztę (html, js etc) przez 24h. W założeniu PWA powinno się tego trzymać.

ksciana commented 2 years ago

Obecna strategia w serviceWorker jest taka, że na sprawdzany jest cache, gdy zawiera odpowiedz na żądanie, jest zwracany, a gdy nie ma w nim jest wysyłane żądanie do serwera. Możne lepiej będzie odwrócić to zachowanie: próba pobrania z serwera, a gdy się powiedzie dodać/zaktualizować. Cache będzie tylko w przypadku braku sieci.

https://web.dev/offline-cookbook/

ttomasz commented 2 years ago

@ksciana może tak być. Zrobisz PR? bo użytkownicy narzekają, że strona się im nie odświeża (liczba pokazywanych aed) bez ctrl+f5