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

Support multiple languages #187

Merged
merged 43 commits into from
Feb 2, 2018
Merged

Support multiple languages #187

merged 43 commits into from
Feb 2, 2018

Conversation

hql287
Copy link
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

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
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
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

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
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

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

@hql287
Copy link
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