-
Notifications
You must be signed in to change notification settings - Fork 3
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
Review front end #40
Review front end #40
Conversation
WilliamHarvey97
commented
Oct 20, 2020
•
edited
Loading
edited
- Rename some files
- Add prop-types where it was lacking
D'après toi @abelfodil , est-ce qu'il faut mettre le texte dans les fichiers text.json ou on devrait laisser faire les text.json et mettre le contenu directement dans le code? |
Si on prévoit de traduire le site, je dirais dans un json, sinon dans le code direct. |
Co-authored-by: Anes Belfodil <abelfodil@users.noreply.github.com>
D'un autre côté me semble la localisation c'est pas un super gros overhead non (en angular en tout cas)? Et c'Est quand même pratique avoir tout le texte dans un dicitonnaire. Ça simplifie la lecture de commits. |
Effectivement, c'est pas un gros overhead, mais ça prend quand même un tantinet plus de temps, ne serait-ce que pour trouver le nom des clés 😆 |