[TECH] Améliorer la propreté du code #373

VincentHardouin · 2022-02-25T16:46:49Z

🦄 Problème

Dans l'IDE pleins d'erreurs sont remontées.

Utiliser les raccourcis css quand c'est possible
Utiliser l'interpolation pour les variables dans la fonction scss calc.
Utiliser le protocole https au lieu d'http dans la page d'erreur.
Réunir la config du plugin i18n.
Utiliser le chemin par défaut des fichiers de prismic permettant de ne plus écrire une config spécifique

Des infos supplémentaires, trucs et astuces ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc.

pix-service · 2022-02-25T16:48:25Z

I'm deploying this PR to these urls:

Please check it out!

VincentHardouin force-pushed the tech-clean-up branch from 1eaf756 to 8bcadab Compare February 25, 2022 16:51

VincentHardouin self-assigned this Feb 25, 2022

VincentHardouin added 👀 Tech Review Needed team-evaluation PR relatives à l'expérience d'évaluation labels Feb 25, 2022

VincentHardouin force-pushed the tech-clean-up branch from 8bcadab to 2ef48b0 Compare March 2, 2022 15:00

yannbertrand approved these changes Mar 4, 2022

View reviewed changes

alexandrecoin approved these changes Mar 9, 2022

View reviewed changes

VincentHardouin added 🚀 Ready to Merge Tech Review OK and removed 👀 Tech Review Needed labels Mar 9, 2022

VincentHardouin added 6 commits March 9, 2022 08:34

sr: use interpolation in calc

7fe4b6a

sr: move required prismic files in default location

ad3cfbe

sr: unify i18n config

4e19007

fix(error-content): use https protocol

fe0bf16

feat(css): use shorthand when possible

c7b0c39

fix(css): use possible weight in button

ce19c8e

pix-service-auto-merge force-pushed the tech-clean-up branch from 2ef48b0 to ce19c8e Compare March 9, 2022 08:34

pix-service-auto-merge merged commit 9a841b1 into dev Mar 9, 2022

pix-service-auto-merge deleted the tech-clean-up branch March 9, 2022 08:35