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

Passage à vite #621

Merged
merged 16 commits into from
Jan 31, 2025
Merged

Passage à vite #621

merged 16 commits into from
Jan 31, 2025

Conversation

ocruze
Copy link
Member

@ocruze ocruze commented Jan 20, 2025

https://symfony-vite.pentatrion.com/

J'ai dû faire une copie "one-shot" des actualités et des favicons dans le dossier public. Les favicons ne changeront pas fréquemment donc pas besoin de gérer les avec vite. Et les actualités ne bougent pas souvent non plus, sachant qu'elles vont disparaître de ce dépôt.

J'avais tenté d'inclure tout ça dans le build avec import.meta.glob, ça marche en dev mais pas en build, du coup j'ai laissé tomber.

Du coup,
Sur linux avec docker :

  • vite dev : les svg ne chargent pas parce qu'apparemment "mixed content", à investiguer, mais pas très grave, le reste du site fonctionne bien y compris le hot reload 🔥
  • vite build : j'ai l'impression que tout marche

Sur windows avec xampp :

  • vite dev : même chose que sur linux
  • vite build : le site est plutôt fonctionnel, juste le préchargement des polices (dans templates/base.html.twig) ne marchent pas

Sur mut-dev :
Ça a l'air de bien fonctionner ✅

Légers changements d'installation :
Doc mise à jour : https://github.com/IGNF/cartes.gouv.fr/blob/feat/use-symfony-vite-bundle/docs/developer/install.md

package.json Show resolved Hide resolved
@ocruze ocruze force-pushed the feat/use-symfony-vite-bundle branch from 3825e66 to d368af6 Compare January 30, 2025 16:27
@ocruze ocruze marked this pull request as ready for review January 30, 2025 16:28
@ocruze ocruze merged commit 969e154 into main Jan 31, 2025
5 checks passed
@ocruze ocruze deleted the feat/use-symfony-vite-bundle branch January 31, 2025 15:03
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.

2 participants