Skip to content

Review front end#40

Merged
WilliamHarvey97 merged 4 commits intomasterfrom
review-front-end
Oct 20, 2020
Merged

Review front end#40
WilliamHarvey97 merged 4 commits intomasterfrom
review-front-end

Conversation

@WilliamHarvey97
Copy link
Copy Markdown
Contributor

@WilliamHarvey97 WilliamHarvey97 commented Oct 20, 2020

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

@WilliamHarvey97
Copy link
Copy Markdown
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?

Comment thread web/src/components/scroll_to_top.js
@abelfodil
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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