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: update Promise.prototype.finally() to original text #17811

Merged
merged 17 commits into from
Mar 18, 2024

Conversation

2chanhaeng
Copy link
Contributor

@2chanhaeng 2chanhaeng commented Jan 9, 2024

Description

Promise.prototype.finally() 페이지를 원문에 맞춰 업데이트 했습니다.

Motivation

Additional details

Related issues and pull requests

#14965

[Typo error in Promise.prototype.finally() page mdn#14869](mdn#14869) 이슈에서 제안했던 내용에 따라 수정했습니다.
@2chanhaeng 2chanhaeng requested a review from a team as a code owner January 9, 2024 07:17
@2chanhaeng 2chanhaeng requested review from sunhpark42 and removed request for a team January 9, 2024 07:17
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Jan 9, 2024
Copy link
Contributor

github-actions bot commented Jan 9, 2024

Preview URLs

Flaws (4)

URL: /ko/docs/Web/JavaScript/Reference/Global_Objects/Promise/finally
Title: Promise.prototype.finally()
Flaw count: 4

  • macros:
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Promise/catch does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/catch
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Promise/catch does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/catch
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Promise/catch does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/catch
  • broken_links:
    • Can use the English (en-US) link as a fallback
External URLs (1)

URL: /ko/docs/Web/JavaScript/Reference/Global_Objects/Promise/finally
Title: Promise.prototype.finally()

(comment last updated: 2024-03-10 22:42:39)

@2chanhaeng
Copy link
Contributor Author

#14965 을 위해 원문에 맞춰 업데이트 했습니다.

@2chanhaeng 2chanhaeng changed the title ko: fix issue #14869 in Promise.prototype.finally() ko: update Promise.prototype.finally() to original text Jan 9, 2024
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.

@sunhpark42 3주가 지나 멘션드렸습니다. 확인 한번 부탁드립니다 :)

@2chanhaeng 님, 문서 갱신 표시도 같이 추가하면 좋을듯합니다!

문서 갱신 표시 안내서에 따라 l10n.sourceCommit 추가 부탁드립니다.

@2chanhaeng
Copy link
Contributor Author

@hochan222 안녕하세요. 리뷰에서 말씀해주신 변경 요청 사항 수정 완료했습니다! 감사합니다.

@sunhpark42
Copy link
Contributor

안녕하세요. 리뷰가 늦어져서 죄송합니다. 명일 2/26일까지 리뷰 드릴 수 있도록 하겠습니다.
기여감사합니다!

Copy link
Contributor

@sunhpark42 sunhpark42 left a comment

Choose a reason for hiding this comment

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

안녕하세요. 리뷰가 늦어졌습니다.
전체 내용중 누락된 부분이 있어서 코멘트 남겨두었습니다. 확인 부탁드립니다.

기여 감사합니다. 🙇‍♀️

@2chanhaeng
Copy link
Contributor Author

@sunhpark42 안녕하세요. 리뷰에서 말씀해주신 변경 요청 사항 수정 완료했습니다! 감사합니다.

Copy link
Contributor

@sunhpark42 sunhpark42 left a comment

Choose a reason for hiding this comment

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

안녕하세요 @2chanhaeng 님, 수정 해주신 부분 확인 했습니다. 감사합니다 :)
오늘도 좋은하루되세요!

@sunhpark42
Copy link
Contributor

auto-merge 눌러놓겠습니다.
@hochan222 님, Request Change 확인 부탁드립니다. :)

@sunhpark42 sunhpark42 enabled auto-merge (squash) March 17, 2024 07:22
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.

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

@sunhpark42 sunhpark42 merged commit 083ffeb into mdn:main Mar 18, 2024
7 checks passed
ciyoe pushed a commit to ciyoe/translated-content that referenced this pull request Apr 20, 2024
* ko: fix error in `Promise.prototype.finally()`

[Typo error in Promise.prototype.finally() page mdn#14869](mdn#14869) 이슈에서 제안했던 내용에 따라 수정했습니다.

* ko: lint `Promise.prototype.finally()` page

* ko: update `Promise.prototype.finally()` to original text

mdn#14965 을 위해 [원문](https://github.com/mdn/content/blob/main/files/en-us/web/javascript/reference/global_objects/promise/finally/index.md)에 맞춰 업데이트 했습니다.

* ko: remove comment in `Promise.prototype.finally()`

번역하면서 남겨둔 주석을 지웠습니다.

* ko: remove headers in `Promise.prototype.finally()`

린터봇의 코멘트 mdn#17811 (comment) 에 따라 일부 헤더를 지웠습니다.

* ko: change words to align with the glossary guide

mdn#17811 (comment)

* ko: add l10n.sourceCommit

mdn#17811 (review)

* ko: update 'Promise/finally' based on review

mdn#17811 (review)
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.

3 participants