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

Review front end #40

Merged
merged 4 commits into from
Oct 20, 2020
Merged

Review front end #40

merged 4 commits into from
Oct 20, 2020

Conversation

WilliamHarvey97
Copy link
Contributor

@WilliamHarvey97 WilliamHarvey97 commented Oct 20, 2020

  • Rename some files
  • Add prop-types where it was lacking

@WilliamHarvey97
Copy link
Contributor Author

WilliamHarvey97 commented Oct 20, 2020

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?

@abelfodil
Copy link
Contributor

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.

William Harvey and others added 2 commits October 20, 2020 10:04
Co-authored-by: Anes Belfodil <abelfodil@users.noreply.github.com>
@WilliamHarvey97 WilliamHarvey97 merged commit 2989830 into master Oct 20, 2020
@WilliamHarvey97 WilliamHarvey97 deleted the review-front-end branch October 20, 2020 18:17
@mateobelanger
Copy link
Contributor

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.

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.

@WilliamHarvey97
Copy link
Contributor Author

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 😆

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.

3 participants