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

Issue 545/lingui translation wrappings #600

Conversation

Tyl13
Copy link
Contributor

@Tyl13 Tyl13 commented May 8, 2023

This is my current work that is done on adding Lingui Translation Wrappings. It includes changes to yarn.lock. New messages are extracted by running yarn extract and then they are compiled to Javascript with yarn compile. I have a few questions about certain sections but if you have anything that you want clarified first, or just more information on what is happening. Please ask. When this is finished, it will resolve #545. I believe there are some questions in the issue too that may provide additional information about Lingui.

src/components/i18n/index.js Outdated Show resolved Hide resolved
lingui.config.js Show resolved Hide resolved
lingui.config.js Outdated Show resolved Hide resolved
@helrond helrond changed the base branch from base to development June 13, 2023 14:02
@HaSistrunk
Copy link
Member

HaSistrunk commented Jun 13, 2023

This is looking really solid! I'm wondering if we need a README update to document/point to documentation on how to implement translations.

@Tyl13 Tyl13 marked this pull request as ready for review June 14, 2023 13:51
@Tyl13
Copy link
Contributor Author

Tyl13 commented Jun 14, 2023

Hopefully nothing was missed and it all looks good, but it should be good to merge.

@helrond helrond changed the title DRAFT: Issue 545/lingui translation wrappings Issue 545/lingui translation wrappings Jun 14, 2023
Copy link
Member

@helrond helrond left a comment

Choose a reason for hiding this comment

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

lgtm 🚀

@helrond helrond merged commit 7c6ea93 into RockefellerArchiveCenter:development Jun 15, 2023
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.

Localization and Internationalization of the UI
3 participants