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

Bsr rpa rework scss js statics #16260

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

rpaoloni-pass
Copy link
Contributor

@rpaoloni-pass rpaoloni-pass commented Feb 10, 2025

But de la pull request

Le premier commit enlève aussi une compilation du scss qu'on faisait en lançant un pod backend alors qu'on utilise pas du tout le css du BO depuis ces pods. cette petite modification permet de gagner 1s sur le lancement d'un pod

Le deuxième commit retire une option pour auto recompiler le scss qui n'était pas utilisée et n'est pas nécessaire avec la mise en bundle

Le troisième met tous les JS (et CSS) dans un ficher bundle.js (respectivement bundle.css) pour réduire drastiquement le nombre d'appels et de chargement de fichiers quand on charge une page du BO. Cette modification règle aussi les problèmes de cache qu'on pouvait avoir quand on met a jour un fichier JS(css) alors que l'utilisateur avait déjà une version de ce fichier en cache.

Vérifications

  • J'ai écrit les tests nécessaires
  • J'ai mis à jour le fichier des plans de tests du portail pro si nécessaire
  • J'ai mis à jour la liste des routes et des titres de pages du portail pro si j'en ai rajouté/modifié ou supprimé une.
  • J'ai relu attentivement les migrations, en particulier pour éviter les locks, et je préviens les équipes Shérif et Data
  • J'ai ajouté des screenshots pour d'éventuels changements graphiques
  • J'ai fait la revue fonctionnelle de mon ticket

@rpaoloni-pass rpaoloni-pass marked this pull request as ready for review February 10, 2025 13:14
@rpaoloni-pass rpaoloni-pass force-pushed the bsr-rpa-rework-scss-js-statics branch 3 times, most recently from 405246e to bb7ff41 Compare February 10, 2025 14:27
@rpaoloni-pass rpaoloni-pass force-pushed the bsr-rpa-rework-scss-js-statics branch from bb7ff41 to ab55725 Compare February 12, 2025 16:46
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 this pull request may close these issues.

1 participant