Skip to content

Conversation

apoorvdeshmukh
Copy link
Contributor

This commit adds localization framework and support for languages
en-us, de-de, fr-fr, es-es, pt-br, ko-kr, zh-cn, zh-tw, it-it, ja-jp, ru-ru.
Localized text added for de-de, fr-fr and es-es. Subsequqnt commits will
add localized text for remaining languages. Translations have been taken
from ODBC sqlcmd.rll.lcl for corresponding language.

This commit adds localization framework and support for
languages en-us, de-de, fr-fr, es-es, pt-br, ko-kr, zh-cn, zh-tw,
it-it, ja-jp, ru-ru. Localized text added for de-de, fr-fr and es-es.
Subsequqnt commits will add localized text for remaining languages.
Translations have been taken from ODBC sqlcmd.rll.lcl for corresponding
language.
@apoorvdeshmukh apoorvdeshmukh marked this pull request as draft February 6, 2023 13:00
This commit adds localized text for remaining language tags such
as it-it, pt-br, ja-jp, ko-kr, zh-cn, zh-tw, ru-ru.
@apoorvdeshmukh apoorvdeshmukh marked this pull request as ready for review February 8, 2023 08:29
Copy link
Collaborator

@shueybubbles shueybubbles left a comment

Choose a reason for hiding this comment

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

🕐

This commit addresses review comments and simplifies the
localizer package to instantiate only one language printer
instance at a time.
@apoorvdeshmukh
Copy link
Contributor Author

Are there any pending comments or suggestions?

Copy link
Collaborator

@stuartpa stuartpa left a comment

Choose a reason for hiding this comment

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

:shipit:

@shueybubbles
Copy link
Collaborator

pls Resolve issues as you fix them. Every comment still shows as "Active"


In reply to: 1427551524

Copy link
Collaborator

@shueybubbles shueybubbles left a comment

Choose a reason for hiding this comment

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

:shipit:

@apoorvdeshmukh apoorvdeshmukh merged commit 52073f7 into main Feb 14, 2023
@apoorvdeshmukh apoorvdeshmukh deleted the apoorvdeshmukh/localization_framework branch February 14, 2023 04:57
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.

3 participants