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

chore: remove lazy-hydrate #97

Closed
wants to merge 0 commits into from
Closed

Conversation

ghusse
Copy link
Contributor

@ghusse ghusse commented Nov 19, 2024

Cette dépendance ne semble pas être utilisée (il faut utiliser le composant <LazyHydrate> pour que ça ait un effet, du moins c'est ce que je comprends).

Cette dépendance est en version bêta dans le package.json, et elle semble ne pas avoir été modifiée depuis au moins 4 ans, ce qui laisse penser qu'elle risque de poser souci pour les mises à jour.

La PR #96 doit être mergée d'abord car elle contient des modifications sur le package-lock qui sont aussi incluses dans cette PR (si je ne le fais pas, dans tous les cas il y aura un conflit lors du merge de la première).

@ghusse
Copy link
Contributor Author

ghusse commented Nov 24, 2024

Ok, j'ai fait une erreur avec le merge de master sur cette branche. Mais je viens de m'apercevoir que c'était bel et bien utilisé sur Home. Je ne devais pas être bien réveillé en faisant la PR.

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