-
Notifications
You must be signed in to change notification settings - Fork 5
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
Adds strings localization #24
Conversation
Why did those images got replaced? |
Generated by 🚫 Danger |
Uh, I forgot to mention it! I move all app Resources (assets catalog, splash screen story board and configuration plist) into Resources group - It makes the project structure cleaner. It also looks like Travis is failing again for some reason.. o.O |
No idea, why travis is failing those build logs are sometimes hell... |
This is really weird, it works just fine on my laptop.. |
Cool, it works with new Xcode, I will squash the CI commits and it's ready to merge. |
Require Xcode 9.1 on Travis
2efca7f
to
3b03d6e
Compare
This PR introduces CZ/EN localization discussed in #22. All hardcoded strings are now replaced with localized strings based on user's phone preferences.
Localization string keys are converted to name-spaced enum to ensure compile-time check of strings validity. The enums are generated with SwiftGen using aggregated Xcode target.
I have also added Danger action, which checks whether both languages are updated together.
✅ The README is updated.