-
-
Notifications
You must be signed in to change notification settings - Fork 565
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
feat(i18n): add missing Polish (and English) translation entries #1669
Conversation
Run & review this pull request in StackBlitz Codeflow. |
✅ Deploy Preview for elk-docs canceled.
|
✅ Deploy Preview for elk-zone ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Adding missing translations (including English). |
input-base | ||
> | ||
<input | ||
v-model="form.fieldsAttributes[i - 1].value" | ||
type="text" placeholder="Content" | ||
type="text" placeholder-text-secondary | ||
:placeholder="$t('settings.profile.appearance.profile_metadata_value')" |
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.
@userquin would you add a comment to this PR showing how you would like we announce these two new key?
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.
The message should be this:
New entries to translate (only included in en
and pl-PL
):
settings.profile.appearance.profile_metadata_label
settings.profile.appearance.profile_metadata_value
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.
We only need to copy the source message and paste it on i18n channel.
No description provided.