Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Éviter la mise en cache des modifications CSS #124

Closed
almet opened this issue Sep 30, 2022 · 4 comments · Fixed by #129
Closed

Éviter la mise en cache des modifications CSS #124

almet opened this issue Sep 30, 2022 · 4 comments · Fixed by #129

Comments

@almet
Copy link
Owner

almet commented Sep 30, 2022

@Gwenael-F dans une autre discussion indique :

Un truc, comme il n'y a pas de versionnage sur les fichiers css, les modifs css ne sont pas prises en compte (sauf pour les gens qui vident le cache régulièrement mais c'est rare).
Pour y remédier, il faudrait juste ajouter ça par exemple aux fichiers (surtout app.css et page.css en fait) : app.css?v=1.12
Et on peut faire évoluer la version au fur et à mesure.
J'ai fait ça sur tout les sites que j'ai codé, ça marche très bien.
C'est possible de faire ça facilement?

@almet
Copy link
Owner Author

almet commented Sep 30, 2022

Effectivement, ce serait bien de trouver une solution pour gérer ça ! Le fait d'utiliser les querystrings me semble un moyen simple, et ça devrait fonctionner facilement.

De manière générale, ça pose la question du déploiement continu. Si c'est le modèle qu'on choisit alors on peut imaginer ajouter le hash du dernier commit de manière automatique dans le querystring.

Je suis plutôt pour faire des modifs localement et pour les déployer à un moment où on pense que c'est le moins risqué, auquel cas on peut imaginer de faire des montées de version de d'utiliser ce numéro de version dans le querystring?

@Gwenael-F
Copy link
Contributor

Carrément, je suis pour la deuxième option!
C'est toi qui le fait cette fois-ci peut-être?
Et tu m'expliqueras ;)

almet added a commit that referenced this issue Oct 5, 2022
And use it when loading the css files in order to not have the css files
cached by the browsers after an update.

Fix #124
@almet
Copy link
Owner Author

almet commented Oct 18, 2022

@Gwenael-F tu valide ?

@Gwenael-F
Copy link
Contributor

Oui, ça m’a l'air top!

almet added a commit that referenced this issue Oct 25, 2022
And use it when loading the css files in order to not have the css files
cached by the browsers after an update.

Fix #124
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants