Skip to content
This repository has been archived by the owner on Oct 26, 2021. It is now read-only.

Add french translation and format files #315

Merged
merged 4 commits into from
Jun 26, 2018

Conversation

PierrickGT
Copy link
Contributor

PR Checklist (check all)

  • Updated CHANGELOG.md to describe the included fixes and changes made
  • Included issue number on the description of the PR
  • Commented on the relevant issue thread about this PR

Issue number

#304

Changelog

@ricburton
Copy link
Member

ricburton commented Jun 20, 2018

🙌 Preview this pull request.

Built with commit 9827e32

https://deploy-preview-315--balance-manager.netlify.com

Copy link
Member

@ricburton ricburton left a comment

Choose a reason for hiding this comment

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

All looks good to me!

@jinchung can you review this please?

(She's on holiday this week.)

@TigerW1998
Copy link

Likewise, looks good. I rechecked the translations and I don't think there's anything that needs to be changed at the moment, but I am open to any suggestions as to stylistic choices or errors!

@ricburton
Copy link
Member

ricburton commented Jun 22, 2018 via email

@@ -0,0 +1,196 @@
{
"translation": {
Copy link
Contributor

Choose a reason for hiding this comment

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

hey @PierrickGT! could you change the indentation of this file to 2 spaces so that it matches all of our other language files?

thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

other than that it looks great to me! 🎉 will merge once indentation is fixed

@PierrickGT
Copy link
Contributor Author

Hi @mikedemarais,
I've fixed the indentation and merged the latest branch into this one.
Hope we can merge soon! 🤞

@jinchung jinchung dismissed mikedemarais’s stale review June 26, 2018 19:30

The issue has been addressed

@jinchung jinchung merged commit 32e13f6 into balance-io:latest Jun 26, 2018
@jinchung
Copy link
Contributor

thanks @PierrickGT ! merged

@jinchung jinchung mentioned this pull request Jun 26, 2018
3 tasks
@jinchung
Copy link
Contributor

also thanks @TigerW1998 !

@PierrickGT PierrickGT deleted the features/french_translation branch June 26, 2018 19:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants