Skip to content

Translation cluster admin overview in Bahasa Indonesia#14156

Merged
k8s-ci-robot merged 11 commits intokubernetes:masterfrom
danninov:translate-cluster-admin-overview
May 6, 2019
Merged

Translation cluster admin overview in Bahasa Indonesia#14156
k8s-ci-robot merged 11 commits intokubernetes:masterfrom
danninov:translate-cluster-admin-overview

Conversation

@danninov
Copy link
Copy Markdown
Contributor

@danninov danninov commented May 4, 2019

This PR addresses #13929

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 4, 2019
@k8s-ci-robot k8s-ci-robot requested review from girikuncoro and irvifa May 4, 2019 12:30
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. language/id Issues or PRs related to Indonesian language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels May 4, 2019
@netlify
Copy link
Copy Markdown

netlify bot commented May 4, 2019

Deploy preview for kubernetes-io-master-staging ready!

Built with commit 6c826cc

https://deploy-preview-14156--kubernetes-io-master-staging.netlify.com

@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels May 4, 2019
Copy link
Copy Markdown
Contributor

@girikuncoro girikuncoro left a comment

Choose a reason for hiding this comment

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

all looking good, major ones are just the broken italic formatting. you should review once more through netlify https://deploy-preview-14156--kubernetes-io-master-staging.netlify.com/docs/concepts/cluster-administration/cluster-administration-overview/

just a heads up, all the links on this page are all broken because we haven't translated the linked pages. but that's ok, so once we complete the relevant translations, the links will be corrected automatically.

great job! 🎉


* [Otorisasi](/docs/reference/access-authn-authz/authorization/) dibedakan dari autentikasi, digunakan untuk mengontrol bagaimana *HTTP call* ditangani.

* [Menggunakan *Admission Controllers*](/docs/reference/access-authn-authz/admission-controllers/) akan menjelaskan *plug-in* yang akan mengintersep permintaan ke server Kubernetes API setelah autentikasi dan otorisasi dilakukan.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

intersep is not indonesian word, perhaps use imported english word or some other Indonesian word?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I was looking at kbbi and found https://kbbi.kemdikbud.go.id/entri/intersep. Kompas also using intersep at their articles.
Maybe the intersep is quite unfamiliar for many, so we can keep the imported english word aside intersep like mengintersep (intercept)

Copy link
Copy Markdown
Contributor

@girikuncoro girikuncoro May 5, 2019

Choose a reason for hiding this comment

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

oh ok glad you found this kemdikbud source, we can refer to this aside from

http://kbbi.web.id
http://kbbi.kamus.pelajar.id

please go ahead 👍

cc @irvifa

@girikuncoro girikuncoro self-assigned this May 5, 2019
@girikuncoro
Copy link
Copy Markdown
Contributor

girikuncoro commented May 5, 2019

@danninov the asterisks format are still broken, probably you can just remove them if tricky to make them working properly

https://deploy-preview-14156--kubernetes-io-master-staging.netlify.com/id/docs/concepts/cluster-administration/cluster-administration-overview/#perencanaan-kluster

edit: see if changing from asterisks to underscore could work

@girikuncoro
Copy link
Copy Markdown
Contributor

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 6, 2019
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: girikuncoro

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

The pull request process is described here

Details 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 approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 6, 2019
@k8s-ci-robot k8s-ci-robot merged commit 9b8b5ae into kubernetes:master May 6, 2019
@danninov danninov deleted the translate-cluster-admin-overview branch May 6, 2019 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/id Issues or PRs related to Indonesian language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants