Correction d'un hash d'intégrité vide pour un fichier CSS #204
Conversation
The problems: * Template tag `dsfr_css` includes `global_css.html` template and passes `INTEGRITY_CSS_ICONS` which is never used by the template. * Meanwhile, the template needs `INTEGRITY_UTILITY_CSS` and never gets it, so it renders an empty integrity hash.
David-Guillot
changed the title
|
Salut @Ash-Crow , visiblement il y a besoin d'une validation (de ta part ?) pour que les Github Workflows de vérification de cette PR s'exécutent. Moyen de regarder quand tu as 2 minutes stp ? (Je ne sais pas si tu as été notifié de cette PR, désolé si ce commentaire est redondant 🫤) |
|
Hello ! J'ai vu la PR mais je n'ai pas encore eu le temps de regarder, désolé... Je fais ça dès que possible. En attendant, est-ce que tu peux passer la |
Done! Et pas d'urgence pour la relecture, j'ai contourné le problème pour l'instant, et mon projet est très très early stage. |
🎯 Objectif
Dans sa version actuelle, le templatetag
dsfr_csspasse une variable inutile dans le contexte de son template, et oublie par contre une variable qui serait utile.La conséquence est que dans le rendu HTML, le hash d'intégrité d'un fichier CSS est vide. C'est un problème mineur car les navigateurs ignorent les hashs vides, mais ce sera plus propre avec cette PR.
🔍 Implémentation