-
Notifications
You must be signed in to change notification settings - Fork 249
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
(chore) Add form engine lib translations #2200
base: main
Are you sure you want to change the base?
Conversation
816a907
to
9e3eb30
Compare
Size Change: +2.69 kB (+0.02%) Total Size: 16.1 MB ℹ️ View Unchanged
|
This PR is dependent on openmrs/openmrs-esm-form-engine-lib#457 that needs to be merged in first and a bump of the lib made. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this, @pirupius. I think Transifex should be the source of truth for non-en
locale keys and strings. I don't expect to see them included in the final diff.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. Let me revert changes to the rest of the files and leave only the en.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's not remove the current translations, as it will impact the implementations using these translations.
We can put these translations for review on transifex, as we are close to a new release candidate.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a valid point because there already implementations using them us inclusive and we're in production. @denniskigen are you comfortable with that?
d94264d
to
9e3eb30
Compare
Requirements
Summary
Adds form-engine-lib translations and updates index.ts to make them available
Screenshots
Related Issue
Other