Skip to content

i18n_set_language_option() sets env var to translate R messages#558

Merged
gadenbuie merged 18 commits intorstudio:masterfrom
rossellhayes:i18n-env-var
Jul 28, 2021
Merged

i18n_set_language_option() sets env var to translate R messages#558
gadenbuie merged 18 commits intorstudio:masterfrom
rossellhayes:i18n-env-var

Conversation

@rossellhayes
Copy link
Copy Markdown
Contributor

This PR expands i18n_set_language_option(). In addition to setting a knitr option to translate the learnr interface, it will also now set an environment variable to translate R messages. This will allow users to receive feedback from R, including gradethis feedback, in their own language (when translations are available).

Comment thread tests/testthat/test-i18n.R Outdated
Comment thread tests/testthat/test-i18n.R Outdated
Comment thread tests/testthat/test-i18n.R Outdated
@rossellhayes rossellhayes requested a review from gadenbuie July 21, 2021 22:00
Comment thread R/i18n.R Outdated
Comment thread R/i18n.R Outdated
@rossellhayes rossellhayes requested a review from gadenbuie July 23, 2021 19:43
Copy link
Copy Markdown
Member

@gadenbuie gadenbuie left a comment

Choose a reason for hiding this comment

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

Looks great! Pending small changes and a NEWS entry this is ready to go 🚀

Comment thread R/i18n.R Outdated
Comment thread R/i18n.R Outdated
Comment thread R/i18n.R Outdated
Comment thread R/i18n.R Outdated
@rossellhayes rossellhayes requested a review from gadenbuie July 28, 2021 16:58
Comment thread R/i18n.R Outdated
Comment thread R/i18n.R Outdated
rossellhayes and others added 2 commits July 28, 2021 14:16
Co-authored-by: Garrick Aden-Buie <garrick@adenbuie.com>
Co-authored-by: Garrick Aden-Buie <garrick@adenbuie.com>
@gadenbuie gadenbuie merged commit f260faa into rstudio:master Jul 28, 2021
@rossellhayes rossellhayes deleted the i18n-env-var branch August 4, 2021 17:18
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