Skip to content

[WIP] Add internationalization as discussed in #17#20

Open
ssirois wants to merge 1 commit intosavoirfairelinux:masterfrom
ssirois:feature/i18n
Open

[WIP] Add internationalization as discussed in #17#20
ssirois wants to merge 1 commit intosavoirfairelinux:masterfrom
ssirois:feature/i18n

Conversation

@ssirois
Copy link
Contributor

@ssirois ssirois commented Apr 14, 2017

Fixes #17 by @ssirois

Changes proposed in this pull request:

  • A translation function to be used where internationalized strings are needed in code
  • A new lang property on RingMe.UI constructor
  • English translation
  • French translation

Status

  • READY
  • HOLD
  • WIP (Work-In-Progress)

How to verify this change

Play with the lang property on the test page and realize that strings are available in both English and French.

Additional notes

This is still WIP.

It is a first iteration to confirm that this is going the proper way. Discussions with people from the Ring project still needs to happen in order to align ourselves with there translation work-flow.

We also need to address #17 (comment)

@ssirois ssirois mentioned this pull request Apr 14, 2017
Translate strings in both English and French.
@ssirois ssirois changed the title Add internationalization as discussed in #17 [WIP] Add internationalization as discussed in #17 Oct 26, 2017
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.

2 participants