Skip to content

[문서] 과제 C29 충돌 해결#1853

Open
taejinchoi-cbnu wants to merge 8 commits into
javascript-tutorial:2026-en-mergefrom
taejinchoi-cbnu:task2-clean
Open

[문서] 과제 C29 충돌 해결#1853
taejinchoi-cbnu wants to merge 8 commits into
javascript-tutorial:2026-en-mergefrom
taejinchoi-cbnu:task2-clean

Conversation

@taejinchoi-cbnu
Copy link
Copy Markdown
Contributor

@taejinchoi-cbnu taejinchoi-cbnu commented May 15, 2026

요약

2회차 과제 C29 충돌 해결 완료하였습니다.

peer review 이후 진행 내용

  • reviewer의 수정 제안 적용 및 commit 정리 완료 (1 commit = 1 file)
  • 누락된 문단 번역 진행 완료
  • 일감 배분 시트에서 사라졌던 2-ui/99-ui-misc/03-event-loop/article.md 에 대한 충돌 해결 작업 완료

수정 파일

  • 2-ui/4-forms-controls/3-events-change-input/1-deposit-calculator/task.md
  • 2-ui/4-forms-controls/3-events-change-input/article.md
  • 2-ui/5-loading/01-onload-ondomcontentloaded/article.md
  • 2-ui/5-loading/02-script-async-defer/article.md
  • 3-frames-and-windows/01-popup-windows/article.md
  • 4-binary/02-text-decoder/article.md
  • 5-network/01-fetch/article.md

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
    • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
    • 맞춤법 검사기로 맞춤법을 확인하셨나요?
    • 마크다운 문법에 사용되는 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`) 등의 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

Copy link
Copy Markdown
Contributor Author

@taejinchoi-cbnu taejinchoi-cbnu May 15, 2026

Choose a reason for hiding this comment

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

해당 파일의 경우 전체 미번역이고 /3-frames-and-windowsindex.md만 번역된 미번역 챕터 같아 일단 충돌 마커만 지우고 commit에 전체 번역은 진행하지 않았습니다.
하지만 따로 번역은 진행해두어서 말씀해주시면 바로 commit 하겠습니다.

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.

05-17 세션에서 충돌 해결 후 파일 하나 번역 괜찮다고 하셔서 번역 진행 예정

@taejinchoi-cbnu
Copy link
Copy Markdown
Contributor Author

@Turtle-Hwan 리뷰 부탁드립니다~

Copy link
Copy Markdown

@Turtle-Hwan Turtle-Hwan left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~! 자잘한 부분들 리뷰해보았어요.

Comment thread 2-ui/4-forms-controls/3-events-change-input/article.md
Comment thread 2-ui/4-forms-controls/3-events-change-input/article.md Outdated
Comment thread 2-ui/4-forms-controls/3-events-change-input/article.md
Comment thread 2-ui/5-loading/01-onload-ondomcontentloaded/article.md Outdated
Comment thread 2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread 2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread 2-ui/5-loading/02-script-async-defer/article.md
Comment thread 4-binary/02-text-decoder/article.md Outdated
Comment thread 5-network/01-fetch/article.md Outdated
Comment thread 5-network/01-fetch/article.md Outdated
@taejinchoi-cbnu
Copy link
Copy Markdown
Contributor Author

2-ui/4-forms-controls/3-events-change-input/article.md 의 ( - 옮긴이) 부분 논의 제외하고는 반영하겠습니다. 꼼꼼한 리뷰 감사합니다.

@taejinchoi-cbnu taejinchoi-cbnu marked this pull request as draft May 17, 2026 10:57
Comment thread 2-ui/5-loading/02-script-async-defer/article.md Outdated
@taejinchoi-cbnu taejinchoi-cbnu changed the title [문서] 2회차 과제 C29 충돌 해결 [WIP] [문서] 2회차 과제 C29 충돌 해결 May 20, 2026
@taejinchoi-cbnu taejinchoi-cbnu marked this pull request as ready for review May 22, 2026 12:18
@taejinchoi-cbnu taejinchoi-cbnu changed the title [WIP] [문서] 2회차 과제 C29 충돌 해결 [문서] 과제 C29 충돌 해결 May 22, 2026
@taejinchoi-cbnu
Copy link
Copy Markdown
Contributor Author

@Turtle-Hwan 코멘트 주신 내용 반영과 commit 정리 진행하였습니다.
최종 리뷰 부탁드릴게요~ 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants