dimitridumont / clear-cache

Simple javascript library to clear cache and reload after

Hi there 👋

I'm Dimitri Dumont, a front-end developper passionate about writing clean code. I'm interested in craftsmanship, clean and well-designed code

About me

💻 I'm a front-end developer freelance
🏍 I am passionate about moto, specialy Harley Davidson & vintage
🏋️‍♂️ I am also passionate about musculation
📚 I like to learn constantly and share
🌍 Laon, France

Technologies

▪ Front-end : React.js | Redux | Next.js | Typescript | Javascript
▪ Back-end : Node.js | SQL | Typescript
▪ Versionning : Git
▪ API implementation: Stripe | Twilio | etc.
▪ Tests : units | integrations | end-to-end
▪ DDD | CQRS | TDD | Clean code | hexagonal architecture | agility
▪ CI/CD: Github actions | Gitlab CI/CD

📖 Latests blog posts

• Dependency inversion in front-end
• Hexagonal architecture in front-end
• Clean code: writing quality code - Part 2
• Clean code: writing quality code - Part 1

🔌 Social

clear-cache's Issues

cache delete not blocking

Without actually testing this case fully, when looking over the code, I think it doesn't block and therefore the reload may happen too early.

Please correct me if I am wrong :-)

I think this

        caches.keys().then((names) => {
            names.forEach(async (name) => {
                await caches.delete(name)
            })
        })

       if (reloadAfterClear) window.location.reload();

should be this (or similar):

caches.keys().then(names => {
  return Promise.all(names.map(caches.delete));
}).then(() => {
  if (reloadAfterClear) window.location.reload();
});

as the forEach does not wait for the await. (Because the function you pass forEach itself is async)
So it has to be a map which returns the promises and then await them all with Promise.all

Add type declarations for TypeScript

Hi @dimitridumont

This is not a necessarily an issue, but rather a friendly request whether you could add / create @types/clear-cache.

I would like to add this repo to one of my projects that is built with TypeScript, and currently I'm seeing this message:

Release new version

@dimitridumont Would you be able to release a new version please to capture my merged MR?