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

[TECH] Supprimer le store Vuex (PIX-1468). #320

Merged
merged 4 commits into from
Nov 8, 2021

Conversation

VincentHardouin
Copy link
Member

@VincentHardouin VincentHardouin commented Nov 5, 2021

🦄 Problème

Nous utilisons le store principalement pour les différents menus qui sont présents sur toutes les pages.
Comme ils sont présents sur toutes les pages, ils ont été mis directement dans le layout. Cela avait pour conséquence qu'à l'époque Nuxt ne proposait pas de fetcher des informations depuis un composant. Cependant depuis la version 2.12 c'est devenu possible.
De plus, entre temps le site est devenu statique, le store l'est aussi devenu par conséquent : Nuxt génère un fichier state.js que nous allons chercher dès la première requête au site. Il n'a donc aucun intérêt.

🤖 Solution

  • Remplacer les appels du store dans les composants nécessaires.

🌈 Remarques

Des infos supplémentaires, trucs et astuces ?

💯 Pour tester

  • Vérifier le bon fonctionne des différentes navigations dans les différentes langues etc.

@VincentHardouin VincentHardouin self-assigned this Nov 5, 2021
@VincentHardouin VincentHardouin added 👀 Tech Review Needed team-evaluation PR relatives à l'expérience d'évaluation labels Nov 5, 2021
@VincentHardouin VincentHardouin force-pushed the tech-remove-vuex-store branch 2 times, most recently from d2a5e37 to 86c04b3 Compare November 5, 2021 14:45
@VincentHardouin VincentHardouin changed the title [FEATURE] Supprimer le store Vuex. [FEATURE] Supprimer le store Vuex (PIX-1468). Nov 5, 2021
@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

Copy link
Contributor

@alexandrecoin alexandrecoin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@VincentHardouin VincentHardouin changed the title [FEATURE] Supprimer le store Vuex (PIX-1468). [TECH] Supprimer le store Vuex (PIX-1468). Nov 6, 2021
Comment on lines +57 to +60
this.mainFooters = await documentFetcher(
this.$prismic,
this.$i18n
).findByType(documents.mainFooter)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion(non bloquant): Je ne sais pas si c'est possible techniquement, est-ce qu'on pourrait faire un plugin pour injecter le documentFetcher ? Comme ça on ferait :

Suggested change
this.mainFooters = await documentFetcher(
this.$prismic,
this.$i18n
).findByType(documents.mainFooter)
this.mainFooters = await this.$documentFetcher.findByType(documents.mainFooter)

cf le 1er exemple où ils injectent un $hello : https://nuxtjs.org/docs/directory-structure/plugins/

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si j'y ai pensé, je pense le faire dans une autre PR !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants