Skip to content

Support multiple languages#187

Merged
hql287 merged 43 commits intodevfrom
multiple-languages
Feb 2, 2018
Merged

Support multiple languages#187
hql287 merged 43 commits intodevfrom
multiple-languages

Conversation

@hql287
Copy link
Copy Markdown
Owner

@hql287 hql287 commented Jan 22, 2018

This is a working demo (Form tab only) which includes translation for English, French and Vietnamese. User can set the default language in settings, hit save and the app's language would be updated immediately.

Please have a look at a translation file format and let me know what you think.

@SkyzohKey Can you review this? Thanks!

Todos:

  • Translate Invoices tab
  • Translate Contacts tab
  • Translate Settings tab
  • Translate Notification
  • Translate Messages
  • Translate Dialogs
  • Translate previewWindow
  • Translate tourWindow
  • Fix Tests

Fix #9

@edmael
Copy link
Copy Markdown

edmael commented Jan 23, 2018

@hql287 hit me up when the translation system is up to date, I'd like to add the italian language.

@hql287
Copy link
Copy Markdown
Owner Author

hql287 commented Jan 23, 2018

Will do! The lovely folks at Crowin just decided to give Manta a free plan as being an open-source project and I’m setting it up at the moment. Will let you know when everything is ready!

@hql287 hql287 mentioned this pull request Jan 23, 2018
@hql287
Copy link
Copy Markdown
Owner Author

hql287 commented Jan 23, 2018

@edmael I just updated #9 with detailed instruction on translating Manta using Crowdin. Please take a look and let me know if you have any issue. Thanks!

@edmael
Copy link
Copy Markdown

edmael commented Jan 23, 2018

Hey @hql287 I don't see the update, but it's easy to find the project on Crowdin and work there so I already did it (part of it, will finish tomorrow).

@hql287
Copy link
Copy Markdown
Owner Author

hql287 commented Jan 23, 2018

@edmael Awesome! 👍 (Sorry I meant issue #9, not this PR :p )

- Added tranlsation for Settings page
- Refactored Form translation
- Move common translation to a separate file
@hql287 hql287 changed the title Support multiple languages [WIP] Support multiple languages Jan 25, 2018
@SkyzohKey
Copy link
Copy Markdown

Reviewed the whole PR everything looks nice to me, LGTM ! 😃

@hql287
Copy link
Copy Markdown
Owner Author

hql287 commented Jan 29, 2018

Thanks, @SkyzohKey. Still have some components left to be refactored though. Will try to get this done asap so people can work on the translation.

@hql287 hql287 changed the title [WIP] Support multiple languages Support multiple languages Jan 31, 2018
@hql287 hql287 merged commit d55fef3 into dev Feb 2, 2018
@hql287 hql287 deleted the multiple-languages branch March 5, 2018 15:05
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