Met à jour les dépendances de l'application

Cette MR dépend de la branche remove-voted-plf : !5 (merged) est à merger avant puis cette MR !6 (merged) sera à placer au regard de la branche master).

Ceci a été réalisé avec @eraviart et comprend :

• La mise à jour des frameworks de l'application et en particulier Svelte et Tailwind (de v2 à v3).
• La mise à jour les syntaxes de requêtes (vers les event based requests).
• La mise à jour les syntaxes d'authentification.

Des informations complémentaires ?

Désormais, le résultat de npm outdated ne renvoie que deux outdates qui n'en sont pas vraiment puisque Current/Wanted sont identiques (mais listés parce qu'en version next) :

Package                        Current          Wanted        Latest  Location                             Depended by
@sveltejs/adapter-node   1.0.0-next.73   1.0.0-next.73  1.0.0-next.0  node_modules/@sveltejs/adapter-node  leximpact-budget-ui
@sveltejs/kit           1.0.0-next.302  1.0.0-next.302  1.0.0-next.1  node_modules/@sveltejs/kit           leximpact-budget-ui

Et npm run build ne renvoie pas d'erreur (éventuellement un warning).

changed the description

@benoit-cty En CI, le build semble échouer en raison de <--- JS stacktrace ---> ... FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory (job)

Dans svelte.config.js, on définit un chunkSizeWarningLimit: 3072 mais je ne pense pas que ce soit lié puisqu'un dépassement devrait générer un warning (pas de fatal error).

As-tu une idée de la source de l'erreur ?

changed the description

assigned to @benoit-cty

C'est réglé : https://git.leximpact.dev/leximpact/leximpact-budget-ui/-/pipelines/3305 Il n'y avait que 1 500 Mo de mémoire sur les serveurs budget, j'ai mis 4 096 Mo et ça passe.

approved this merge request

assigned to @sandcha and unassigned @benoit-cty

deleted the remove-voted-plf branch. This merge request now targets the master branch

Rebase en cours

added 13 commits

• 8194ec61...bdf11113 - 2 commits from branch master
• 3e24d968 - Upgrade dependencies
• 7fc208eb - Fix config.kit.target error
• a25435b0 - Upgrade from tailwind v2 to v3
• 06fd06a4 - Upgrade to new JS event based requests
• 4770e815 - Remove unused NavBarBudget component
• 6f95b603 - Upgrade to new syntax
• 713b3e79 - Remove openfisca parameters dir from metadata
• 14cb64e0 - Auto-format files
• 7f8d39fe - Add budget configuration to session interface
• 98fc210b - Fix some linting
• 241c1f8d - Allow for different fetch when app is running and test

Compare with previous version

À l'exécution de npm run build, il reste une erreur o_o : > Using @sveltejs/adapter-node TypeError: Cannot destructure property 'headers' of 'event.request' as it is undefined.

Ceci me semble probablement en lien avec le commit 06fd06a4.

added 2 commits

• 557e822e - Init a CHANGELOG
• 5f755290 - Bump version for the first time to 1.0.1

Compare with previous version

merged

mentioned in commit 71c584ff

Après avoir exécuté les commandes suivantes, l'erreur TypeError n'a plus lieu :

nvm install 16
npm install
npm run build
npm run preview

À ce stade, on attribue le changement au temps écoulé depuis l'ouverture de la MR (donc des évolutions de librairies ont eu lieu) .