-
Notifications
You must be signed in to change notification settings - Fork 4
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
Smartling translations are completed for commit cbbcd7fb57679c3c953ca951fd90e96dcb492cd6 from main #1238
Smartling translations are completed for commit cbbcd7fb57679c3c953ca951fd90e96dcb492cd6 from main #1238
Conversation
…ted to ko-KR locale
…ted to zh-CN locale
…ted to ja-JP locale
…ted to ko-KR locale
…ted to zh-CN locale
…ted to ja-JP locale
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@bzavhorodskyi Are you able to request translation for this PR for all locales again? I don't know why, but strings are shown as "Awaiting Authorization" in Smartling but jobs were not created. |
@tomokogoto1 https://dashboard.smartling.com/app/projects/224fc9b33/account-jobs/224fc9b33:qejsrfcuuit6 I`ve added these strings to the job |
…ted to ko-KR locale
…ted to zh-CN locale
…ted to ja-JP locale
…ted to ko-KR locale
…ted to zh-CN locale
…ted to ja-JP locale
@tomokogoto1 New translations were added here |
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.
https://dev-docs-moemy7vyk-product-content.vercel.app/en/sdk/web
Translation is missing for all non-EN locales. Could you check why?
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.
@bzavhorodskyi This page has missing translation (sub-heading and small texts)
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.
@bzavhorodskyi This article contains missing translation
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.
@bzavhorodskyi Same as other locales, some translation is missing
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.
@bzavhorodskyi Same as other locales, some translation is missing
@tomokogoto1 For some reason, some words for this file were missed, I`ve added these strings to the job: https://dashboard.smartling.com/app/projects/224fc9b33/account-jobs/224fc9b33:qejsrfcuuit6
…ted to ko-KR locale
…ted to zh-CN locale
…ted to ja-JP locale
…ted to ko-KR locale
…ted to zh-CN locale
…ted to ja-JP locale
Re: https://dev-docs-879v3d9ub-product-content.vercel.app/ko/sdk/web
|
…ing-translation-completed-qejsrfcuuit6-from-main
@tomokogoto1 Fixed: https://dev-docs-284fta94j-product-content.vercel.app/ja/sdk/web |
Thank you for fixing: #1238 (comment) Looks good! |
Please review this translation pull request. The relevant Smartling translation job is available here - https://dashboard.smartling.com/app/accounts/aaa8fc6f/account-jobs/224fc9b33:qejsrfcuuit6
Here is a link to the source commit which triggered translation - cbbcd7f