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

Remove or fully apply i18n. #77

Open
lieuwex opened this issue Dec 9, 2014 · 11 comments
Open

Remove or fully apply i18n. #77

lieuwex opened this issue Dec 9, 2014 · 11 comments

Comments

@lieuwex
Copy link
Member

lieuwex commented Dec 9, 2014

Don't implement it for just 1/8th of the app, it just doesn't make sense.

@tkon99
Copy link
Member

tkon99 commented Dec 12, 2014

Fully apply +1

@tkon99
Copy link
Member

tkon99 commented Dec 12, 2014

Let's make it English/Dutch for the time being

@lieuwex
Copy link
Member Author

lieuwex commented Dec 12, 2014

If we grow; http://Crowdin.com/

@tkon99
Copy link
Member

tkon99 commented Dec 13, 2014

Great solution indeed

@tkon99
Copy link
Member

tkon99 commented Jan 3, 2015

http://zanata.org/
Better solution

@tkon99 tkon99 added this to the Production-ready milestone Jan 17, 2015
@lieuwex lieuwex removed their assignment Jun 22, 2015
@tkon99 tkon99 removed this from the Production-ready milestone Aug 29, 2015
@lieuwex
Copy link
Member Author

lieuwex commented Mar 28, 2016

This will also help to have consistent strings for the same error, like 'Wachtwoord is fout' for each 'wrong-password'.

@tkon99
Copy link
Member

tkon99 commented Aug 29, 2016

What's the status on this?

@lieuwex
Copy link
Member Author

lieuwex commented Aug 29, 2016

None. :p
But I'm not sure if this is a high priority right now, I can set up the
foundation for it though, so that we can slowly convert the app to use
i18n.

On Mon, 29 Aug 2016, 12:22 Thomas Konings, notifications@github.com wrote:

What's the status on this?


You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
#77 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFvYKZu0F7fJhTzPOR6xoKn_h60srcOaks5qkrLtgaJpZM4DGTCL
.

@tkon99
Copy link
Member

tkon99 commented Aug 29, 2016

I think a foundation wouldn't be out of place but this is about as low as the priority can get.

@lieuwex
Copy link
Member Author

lieuwex commented Aug 29, 2016

I rebased the very old i18n branch to master, so we can add strings to it there.

@lieuwex
Copy link
Member Author

lieuwex commented Aug 30, 2016

I have merged the i18n branch into develop with 66e29b1, we're now using that instead of the strings in Local, so we have a beginning :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants