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

Auth pages i18n #1913

Merged
merged 1 commit into from Dec 12, 2020
Merged

Auth pages i18n #1913

merged 1 commit into from Dec 12, 2020

Conversation

ghys
Copy link
Member

@ghys ghys commented Dec 10, 2020

This implements localized messages for the authorize, change
password and create API token pages using a resource bundle.

Messages in English & French are included.

Signed-off-by: Yannick Schaus github@schaus.net

This implements localized messages for the authorize, change
password and create API token pages using a resource bundle.

Messages in English & French are included.

Signed-off-by: Yannick Schaus <github@schaus.net>
@ghys
Copy link
Member Author

ghys commented Dec 10, 2020

@kaikreuzer after this is approved & merged, thanks in advance to add these properties files in Crowdin as we discussed :)

@ghys
Copy link
Member Author

ghys commented Dec 12, 2020

Would appreciate an expedited review of this so it can be included in the release :)
Given that the UI was translated into 15 languages in less 72 hours by the community, it could well happen for these too, and it would add to a good experience.

image

And if the properties files are configured it's relatively harmless to let the translation campaign happen next week for these IMO. If there are e.g. file encoding issues this can be adjusted before merging.

Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kaikreuzer kaikreuzer merged commit 67bdfa3 into openhab:master Dec 12, 2020
@kaikreuzer kaikreuzer added this to the 3.0.0.RC1 milestone Dec 12, 2020
@kaikreuzer kaikreuzer added the enhancement An enhancement or new feature of the Core label Dec 12, 2020
@kaikreuzer
Copy link
Member

@ghys I have included these files in the crowdin config as discussed!

splatch pushed a commit to ConnectorIO/copybara-hab-core that referenced this pull request Jul 11, 2023
This implements localized messages for the authorize, change
password and create API token pages using a resource bundle.

Messages in English & French are included.

Signed-off-by: Yannick Schaus <github@schaus.net>
GitOrigin-RevId: 67bdfa3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature of the Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants