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

[ko] Translate 'How To Contribute' into Korean #747

Merged
merged 2 commits into from
Apr 21, 2022

Conversation

jihoon-seo
Copy link
Collaborator

@jihoon-seo jihoon-seo commented Apr 15, 2022

Relevant with #379 (comment)

참고: 영어 원문 파일은 dev-ko 의 것이 아니라 main 의 것을 이용했습니다.

Preview: https://deploy-preview-747--cncfglossary.netlify.app/ko/contribute/

@netlify
Copy link

netlify bot commented Apr 15, 2022

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit e76046d
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/6260fd70b519560009c372fb
😎 Deploy Preview https://deploy-preview-747--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.


여러 템플릿이 보일 것이다. 새로운 영문 용어를 제안하려면, "Request to add a new term (English)" 를 선택한다.

![템플릿](/images/how-to/english-issue-template.jpg)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Preview에서 이 이미지만 404 Not found로 보입니다.

Netlify preview instance에 이 jpg 파일이 제대로 올라가지 않아서 그런 것으로 추정됩니다.
dev-ko 내지는 main에 merge되면 정상적으로 보일 것 같습니다.

Copy link
Collaborator

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

@jihoon-seo 리뷰의견 드렸습니다! :)

content/ko/contribute/_index.md Outdated Show resolved Hide resolved
content/ko/contribute/_index.md Outdated Show resolved Hide resolved
content/ko/contribute/_index.md Outdated Show resolved Hide resolved
content/ko/contribute/_index.md Outdated Show resolved Hide resolved
content/ko/contribute/_index.md Outdated Show resolved Hide resolved
content/ko/contribute/_index.md Outdated Show resolved Hide resolved
@jihoon-seo
Copy link
Collaborator Author

@seokho-son 리뷰 감사합니다. 😊
주신 의견을 모두 반영하였으며, 두 번째 커밋에서 확인하실 수 있습니다.

변경 사항을 쉽게 확인하실 수 있도록 squash는 아직 하지 않았으며, 필요한 경우 진행하겠습니다.
또는 "Squash and merge" 하시면서, commit 메시지의 (제목만으로도 충분할 것이므로) 내용은 모두 삭제하거나 적절히 정리하는 방법도 있겠습니다. 😊

Copy link
Collaborator

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

LGTM :)

@yunkon-kim yunkon-kim self-assigned this Apr 21, 2022
Copy link
Collaborator

@yunkon-kim yunkon-kim left a comment

Choose a reason for hiding this comment

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

lgtm

@yunkon-kim yunkon-kim merged commit 35f58b6 into cncf:dev-ko Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/ko for Korean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants