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

fix: messages of missing/incorrect locale cannot be translated #915

Merged
merged 1 commit into from
Nov 26, 2018

Conversation

takahashim
Copy link
Contributor

Congrats to release 4.1.0 🎉

By the way, to translate error messages, the words "incorrect" and "missing" in
com/adobe/epubcheck/tool/EpubChecker.java should be in messages.properties.

To translate error messages, the words "incorrect" and "missing" in
com/adobe/epubcheck/tool/EpubChecker.java should be in messages.properties.
@rdeltour
Copy link
Member

Good catch, thanks @takahashim!

@rdeltour rdeltour self-requested a review November 26, 2018 17:17
@tofi86 tofi86 added this to the 4.1.1 milestone Nov 26, 2018
@tofi86 tofi86 added the type: improvement The issue suggests an improvement of an existing feature label Nov 26, 2018
@tofi86
Copy link
Collaborator

tofi86 commented Nov 26, 2018

Good catch @takahashim!

Added to the new 4.1.1 milestone.

Please also note the current translation updates on branch l10n-update-from-transifex-4.1.0 (PR #914).

@tofi86
Copy link
Collaborator

tofi86 commented Nov 26, 2018

@takahashim Once @rdeltour merged this feature to master it needs half a day until the properties file changes are synced to Transifex.

@rdeltour rdeltour merged commit 91fac12 into w3c:master Nov 26, 2018
@rdeltour
Copy link
Member

(merged, I was waiting for the CI build to succeed ;-)

@takahashim takahashim deleted the incorrect_locale branch November 26, 2018 17:39
@takahashim
Copy link
Contributor Author

Thanks to merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: improvement The issue suggests an improvement of an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants