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

Mise à jour majeure de Sveltekit #378

Closed
6 tasks
Volubyl opened this issue Aug 22, 2022 · 1 comment · Fixed by #427
Closed
6 tasks

Mise à jour majeure de Sveltekit #378

Volubyl opened this issue Aug 22, 2022 · 1 comment · Fixed by #427
Assignees

Comments

@Volubyl
Copy link
Collaborator

Volubyl commented Aug 22, 2022

Description du besoin

Une mise à jour importante de SvelteKit est sortie récement.

Cette mise à jour peut-être assez tricky à bien effectuer car certains fichiers doivent être renommer, de la config doit être changée etc

Pour avoir un aperçu il y a un Migration guide

Ce serait important d'effectuer cette migration asap parce que le documention de svelte kit prend en compte ces changements et comme la documentation n'est pas versionnée c'est devenu un poil compliqué de s'y retrouver ....

Points d'attention

la structure du dossier routes change

Avant dans le dossier route pouvait avoir cette structure

|- blog
|-- index.svelte // Route /blog

Dans l'index.svelte, il pouvait y avoir une fonction load pour le chargement des données avant l'affichage de la page.

Avec la MAJ, la structure de fichier est la suivante

|- blog
|-- index.svelte // Route /blog
|-- index.ts

Le fichier index.ts contient la fonction load qui était avant incluse dans le fichier index.svelte

Procédure

  • Veiller à ce que svelte kit soit bien au moins à la version @sveltejs/[email protected] (en regardaant le package-lock.json)
  • lancer npx svelte-migrate routes
  • regarder là où le script à rajouter des "@migrations" (ce sont les endroits où un peu de travail manuel est nécessaire)
  • lancer en local
  • lancer les tests e2e pour voir si tout est ok
  • déployer sur staging
@Volubyl Volubyl added the front label Aug 22, 2022
@Volubyl
Copy link
Collaborator Author

Volubyl commented Aug 24, 2022

Au niveau calendrier, je propose de faire cete MAJ APRES avoir fini l'intégration de Datapass

Voir #373

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

Successfully merging a pull request may close this issue.

2 participants