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] Améliorer la lisibilité des configurations des modules #441

Merged
merged 7 commits into from
Dec 19, 2022

Conversation

VincentHardouin
Copy link
Member

@VincentHardouin VincentHardouin commented Dec 15, 2022

🦄 Problème

Actuellement, nous configurons les modules que nous utilisons de 2 manières différentes, ce qui créé de la complexité à la lecture et de la confusion lors de l'ajout d'un nouveau module

🤖 Solution

  • Configurer un module en utilisant sa clé de configuration dédiée

🌈 Remarques

  • Certaines clés ont été mises en dessous de là où le module a été déclaré

💯 Pour tester

  • Vérifier le bon fonctionnement des RA

@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

@VincentHardouin VincentHardouin added the team-evaluation PR relatives à l'expérience d'évaluation label Dec 15, 2022
Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

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

Ce rangement me parait aussi plus clair, bien que je ne connaisse pas assez Nuxt pour pouvoir bien juger, et J'ai vérifié fonctionnellement qu'il n'y avait pas de régression.

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 Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants