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: add translation for tools_and_testing/client-side_js_frameworks/react_interactivity_events_state #14898

Conversation

IMHOJEONG
Copy link
Contributor

@IMHOJEONG IMHOJEONG commented Aug 4, 2023

Description

tools_and_testing/client-side_js_frameworks/react_interactivity_events_state를 번역하였습니다.
영어 그대로 해석해서 쓰기에는, 읽는 사람들이 제대로 이해하지 못할 부분도 있어서, 개발에서 사용되는 단어, 문장을 최대한 반영해서 번역하였습니다.

Motivation

React에 익숙하지 않을 때, 처음보면 좋을 가이드라인을 만들면 좋겠다 싶어 이 부분의 번역을 진행하고 있습니다.
좋은 가이드 문서가 있다면, 그만큼 더욱 기술에 친근하게 다가갈 수 있는 부분이라고 생각합니다.

Additional details

정확하지 않은 부분도 있고, 번역하면서 약간 갸우뚱하던 부분들도 존재합니다.
JavaScript나 React와 같이 언어 특성에 따른 단어(state,props)와 문장은 오히려 번역이 아니라, 원문 그대로 참조하여 쓰는 게 더 정확하다고 생각했습니다. (e.g. props를 속성들로 번역해서 사용하는 게 맞는건지에 대한 의문이 들었습니다.)

부족한 부분이 있을 수 있으니, 피드백 주시면 참고해서 빠른 시간에 수정하도록 하겠습니다.

Related issues and pull requests

@IMHOJEONG IMHOJEONG requested a review from a team as a code owner August 4, 2023 02:46
@IMHOJEONG IMHOJEONG requested review from pje1740 and removed request for a team August 4, 2023 02:46
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Aug 4, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2023

Preview URLs

Flaws (117)

URL: /ko/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_events_state
Title: React 상호 작용: 이벤트 및 상태
Flaw count: 117

  • macros:
    • /ko/docs/Learn/HTML/Introduction_to_HTML/Advanced_text_formatting does not exist but fell back to /en-US/docs/Learn/HTML/Introduction_to_HTML/Advanced_text_formatting
    • /ko/docs/Learn/HTML/Introduction_to_HTML/Document_and_website_structure does not exist but fell back to /en-US/docs/Learn/HTML/Introduction_to_HTML/Document_and_website_structure
    • /ko/docs/Learn/HTML/Introduction_to_HTML/Debugging_HTML does not exist but fell back to /en-US/docs/Learn/HTML/Introduction_to_HTML/Debugging_HTML
    • /ko/docs/Learn/HTML/Introduction_to_HTML/Marking_up_a_letter does not exist but fell back to /en-US/docs/Learn/HTML/Introduction_to_HTML/Marking_up_a_letter
    • /ko/docs/Learn/HTML/Multimedia_and_embedding/Images_in_HTML does not exist but fell back to /en-US/docs/Learn/HTML/Multimedia_and_embedding/Images_in_HTML
    • and 112 more flaws omitted
External URLs (1)

URL: /ko/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_events_state
Title: React 상호 작용: 이벤트 및 상태

(comment last updated: 2023-08-24 01:09:23)

@IMHOJEONG IMHOJEONG changed the title ko: add translation for 'tools_and_testing/client-side_js_frameworks/react_interactivity_events_state` ko: add translation for tools_and_testing/client-side_js_frameworks/react_interactivity_events_state Aug 4, 2023
Copy link
Collaborator

@pje1740 pje1740 left a comment

Choose a reason for hiding this comment

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

긴 글 작업하시느라 정말 고생 많으셨습니다! 제가 몇가지 코멘트 남겼는데, 꼭 정답은 아니기 때문에 궁금하신 점은 편하게 질문 남겨주시고, ko locale 차원에서의 논의가 필요하다고 생각되시면 그것도 코멘트 남겨주시면 감사하겠습니다.
제가 코멘트 드린 부분은 크게 이렇습니다.

  • 번역의 어감
  • glossary 원칙에 따른 수정 (event listener 등)
  • React의 고유한 개념의 원문 유지 여부

여러번 반복되는 것들은 하나하나 코멘트를 달지 않았지만 수정은 일괄적으로 부탁드리겠습니다!

@IMHOJEONG
Copy link
Contributor Author

@pje1740 님 안녕하세요.
알려주신 대로, 최대한 피드백을 반영해 보았습니다.

어색하거나, 이상한 부분 있으면 가감없이 주시면 감사하겠습니다!!

@github-actions
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Aug 23, 2023
@IMHOJEONG IMHOJEONG force-pushed the translation/client-side_js_frameworks/react_interactivity_events_state branch from 003c355 to c260bd2 Compare August 23, 2023 13:21
@github-actions github-actions bot removed the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Aug 23, 2023
@IMHOJEONG IMHOJEONG force-pushed the translation/client-side_js_frameworks/react_interactivity_events_state branch from ef701ff to fdc9b9e Compare August 23, 2023 13:41
@IMHOJEONG IMHOJEONG requested a review from pje1740 August 23, 2023 13:47
@pje1740 pje1740 force-pushed the translation/client-side_js_frameworks/react_interactivity_events_state branch from fdc9b9e to aec7d23 Compare August 24, 2023 01:06
Copy link
Collaborator

@pje1740 pje1740 left a comment

Choose a reason for hiding this comment

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

계속 변경이 발생해서 github에서 제공하는 rebase merge 로 우선 컨플릭 제가 해결했습니다! 너무 고생 많으셨습니다!!

@pje1740 pje1740 merged commit e5ba491 into mdn:main Aug 24, 2023
7 checks passed
@IMHOJEONG
Copy link
Contributor Author

@pje1740 님 안녕하세요.

ㅠㅠ 제가 해결을 못해서, 결국 해주셨군요.. 감사합니다!

하나 여쭈어 보고 싶은 부분이 있는데요.
이렇게 변경이 많이 발생하는 부분에 대해서 merge를 진행할 때, rebase 규칙은 어떻게 설정하시고,
conflict을 해결하시는지 궁금합니다.

@pje1740
Copy link
Collaborator

pje1740 commented Aug 24, 2023

기본적으로는 main 브랜치를 현재 작업 중인 브랜치에서 rebase 하고, 해당 변경 사항을 force-push 하여 해결하고 있습니다. 깃헙에서도 update branch 기능을 제공하고 있어서 제가 대단한 걸 한 건 아니고, 이 기능 활용했습니다! 😂
그래도 항상 기여자 분께 1차로 부탁을 드리는 건, 변경사항에 대해 더 잘 인지하고 계시고 누락되는 내용이 없게끔 하기 위함입니다. 다만 최근에는 변화가 잦은 상황이라 리뷰어들이 rebase 하는 경우도 많으니 참고 부탁드립니다!

@IMHOJEONG IMHOJEONG deleted the translation/client-side_js_frameworks/react_interactivity_events_state branch August 24, 2023 01:40
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