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

[hi]Localize content/en/docs/reference/glossary/app-container.md #36063

Closed

Conversation

taniaduggal
Copy link
Contributor

Fixes #35941

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 18, 2022
@k8s-ci-robot k8s-ci-robot added the language/hi Issues or PRs related to Hindi language label Aug 18, 2022
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign mittalyashu for approval by writing /assign @mittalyashu in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Aug 18, 2022
Copy link
Contributor

@divya-mohan0209 divya-mohan0209 left a comment

Choose a reason for hiding this comment

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

Few suggestions, hope you find them useful.

content/hi/docs/reference/glossary/app-container.md Outdated Show resolved Hide resolved
tags:
- workload
---
एप्लिकेशन {{<glossary_tooltip text="पात्र" term_id="container" >}} (या ऐप कंटेनर) {{<glossary_tooltip text="पॉड" term_id="pod" >}} में, {{<glossary_tooltip text ="इनिट कंटेनर" term_id="init-container" >}} स्टार्टअप पूरा होने के बाद शुरू होता है।
Copy link
Contributor

Choose a reason for hiding this comment

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

Why have we used पात्र as the text here when we're using container in its machine translated form everywhere else?

content/hi/docs/reference/glossary/app-container.md Outdated Show resolved Hide resolved
<!--more-->

एक इनिट कंटेनर के साथ, संपूर्ण
आप {{< glossary_tooltip text="कार्यभार" term_id="workload" >}} के लिए महत्वपूर्ण आरंभीकरण विवरण अलग कर सकते हैं।
Copy link
Contributor

Choose a reason for hiding this comment

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

Still not sure about कार्यभार and I presume you meant app workload

Suggested change
आप {{< glossary_tooltip text="कार्यभार" term_id="workload" >}} के लिए महत्वपूर्ण आरंभीकरण विवरण अलग कर सकते हैं।
ऐप {{< glossary_tooltip text="कार्यभार" term_id="workload" >}} के लिए महत्वपूर्ण आरंभीकरण विवरण अलग कर सकते हैं।

Copy link
Contributor

Choose a reason for hiding this comment

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

If it was intended to mean you, in a respectful way, then the structure of this sentence needs to be modified as below.

एक इनिट कंटेनर के साथ, आप संपूर्ण {{< glossary_tooltip text="कार्यभार" term_id="workload" >}} के लिए महत्वपूर्ण आरंभीकरण विवरण अलग कर सकते हैं।

@taniaduggal
Copy link
Contributor Author

@divya-mohan0209 Please take a look. I made suggested changes

__
sema-logo  Summary: 🛠️ This code needs a fix

@sftim sftim deleted the branch kubernetes:dev-1.22-hi.1 November 19, 2022 14:24
@sftim sftim closed this Nov 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/hi Issues or PRs related to Hindi language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants