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 intersection_observer_api #16733

Merged
merged 22 commits into from
Nov 29, 2023

Conversation

YDayeon
Copy link
Contributor

@YDayeon YDayeon commented Oct 28, 2023

Description

/ko/docs/Web/API/Intersection_Observer_API 문서를 한국어로 번역하였습니다.

Motivation

해당 문서의 한국어 번역이 존재하지 않아 번역하게 되었습니다.

@YDayeon YDayeon requested a review from a team as a code owner October 28, 2023 00:30
@YDayeon YDayeon requested review from hochan222 and removed request for a team October 28, 2023 00:30
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Oct 28, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 29, 2023

Preview URLs

Flaws (19)

URL: /ko/docs/Web/API/Intersection_Observer_API
Title: Intersection Observer API
Flaw count: 19

  • macros:
    • /ko/docs/Web/API/IntersectionObserverEntry does not exist but fell back to /en-US/docs/Web/API/IntersectionObserverEntry
    • /ko/docs/Web/API/IntersectionObserverEntry/isIntersecting does not exist but fell back to /en-US/docs/Web/API/IntersectionObserverEntry/isIntersecting
    • /ko/docs/Web/API/IntersectionObserverEntry does not exist but fell back to /en-US/docs/Web/API/IntersectionObserverEntry
    • /ko/docs/Web/API/IntersectionObserverEntry/rootBounds does not exist but fell back to /en-US/docs/Web/API/IntersectionObserverEntry/rootBounds
    • /ko/docs/Web/API/IntersectionObserverEntry/isIntersecting does not exist but fell back to /en-US/docs/Web/API/IntersectionObserverEntry/isIntersecting
    • and 11 more flaws omitted
  • broken_links:
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
External URLs (2)

URL: /ko/docs/Web/API/Intersection_Observer_API
Title: Intersection Observer API

(comment last updated: 2023-11-29 03:56:48)

Copy link
Member

@hochan222 hochan222 left a comment

Choose a reason for hiding this comment

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

@YDayeon 님 안녕하세요.

기여해주셔서 감사합니다.

현재 "[ko] 한국 MDN 문서에 존재하는 레거시 문서 최신화" 이슈에서 레거시 문서들을 최신화하는 작업을 진행하고 있습니다. 해당 이슈에서 일부만 번역된 문서들을 /docs/ko/fully_untranslated_files 폴더로 옮겨두었는데요. 현재 번역주신 파일 /docs/ko/fully_untranslated_files/Web_API_Intersection_Observer_API.md에 존재해서 해당 PR에서 같이 삭제 부탁드립니다.

리뷰 사항 하기에 남겨두었습니다.

고생 많으셨습니다. 좋은 하루 보내세요 :)

files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
files/ko/web/api/intersection_observer_api/index.md Outdated Show resolved Hide resolved
@YDayeon
Copy link
Contributor Author

YDayeon commented Nov 4, 2023

수정 사항 모두 반영해서 다시 커밋했습니다!

@YDayeon YDayeon requested a review from hochan222 November 6, 2023 14:55
Copy link
Member

@hochan222 hochan222 left a comment

Choose a reason for hiding this comment

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

LGTM. 고생 많으셨습니다. 좋은 하루 보내세요 :)

@hochan222
Copy link
Member

hochan222 commented Nov 29, 2023

첫번째 리뷰 메세지에는 PR 삭제를 요청하셨는데, 다음 메세지 리스트에서는 수정 관련 요청을 남겨주셔서 저는 수정 후에 다시 요청을 하라는 뜻으로 해석해서 수정 후에 다시 커밋을 하여 머지를 기다리고 있는 상황입니다. 혹시 제가 이해한 내용이 맞는지, 아니면 PR 삭제를 해야 하는 상황인지 문의 드립니다.

@YDayeon 님, 먼저 휴가와 코로나로 답변이 늦어져서 죄송합니다. (3주나 지나서 정말 죄송합니다 ㅜㅜ..)

메일로 문의 주신 내용에 대해 답변 드립니다. 아래 문장의 의미는

"현재 번역주신 파일 https://github.com/mdn/translated-content/pull/16733#pullrequestreview-1702954841에 존재해서 해당 PR에서 같이 삭제 부탁드립니다."

일반적으로 기여 주시는 파일들은 files/ko 폴더에 있지만, 최근 레거시 문서 제거 작업으로 전체가 번역되지 않은 파일들은 files/ko 폴더에서 /docs/ko/fully_untranslated_files 폴더로 옮겨 놓았습니다. 현재 기여주신 페이지가 https://github.com/mdn/translated-content/blob/main/docs/ko/fully_untranslated_files/Web_API_Intersection_Observer_API.md 경로 즉 /docs/ko/fully_untranslated_files 폴더 내에도 중복해서 존재하고 있어서, 해당 경로의 파일 제거에 대한 요청이었습니다.

현재 기여 주신 PR에 대한 파일은 문제 없습니다..! 리뷰 내용 잘 반영해주셔서 현재 PR 머지하고, 해당 경로의 파일은 제가 다른 PR에서 제거하도록 하겠습니다.

기여해주셔서 감사합니다. 좋은 하루 보내세요 :)

@hochan222 hochan222 merged commit 5127d28 into mdn:main Nov 29, 2023
7 checks passed
@hochan222
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ko Issues related to Korean content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants