-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[ja]Addressed the translation omissions #5983
[ja]Addressed the translation omissions #5983
Conversation
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.
Thank you for your contribution, @shukawam! I added minor requests but other than those, LGTM.
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.
LGTM
a259e38
to
679e3ef
Compare
Hi @ymotongpoo - as mentioned in https://cloud-native.slack.com/archives/C076RUAGP37/p1738265105433829
When |
679e3ef
to
8c8b1d3
Compare
Thank you for your contribution! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey |
@chalin thank you for sharing the process. I could not catch up with the message that mentions the |
#5982
I have addressed the translation omissions in the exporter section.
There are still some English parts within the shortcode, but I plan to handle those during the translation of other sections that use the shortcode.
Executed
npm run fix:all
in my local env and confirmed that there are no issues.