Skip to content

[문서] C28 충돌 해결 및 미번역분 번역#1854

Open
gusdn3477 wants to merge 17 commits into
javascript-tutorial:2026-en-mergefrom
gusdn3477:gusdn3477/sync-c28
Open

[문서] C28 충돌 해결 및 미번역분 번역#1854
gusdn3477 wants to merge 17 commits into
javascript-tutorial:2026-en-mergefrom
gusdn3477:gusdn3477/sync-c28

Conversation

@gusdn3477
Copy link
Copy Markdown

@gusdn3477 gusdn3477 commented May 15, 2026

요약

  • en 저장소에서 업데이트된 내용을 ko 저장소로 업데이트하는 과정에서 발생한 충돌을 해결하였습니다.
  • 커밋된 히스토리를 타고 들어가며 수정된 내용에 대해 번역을 진행하였습니다.
  • 충돌 해결 및 번역 범위
5-network/01-fetch/article.md
5-network/02-formdata/article.md
5-network/04-fetch-abort/task.md
5-network/05-fetch-crossorigin/article.md
5-network/10-long-polling/article.md
5-network/11-websocket/article.md
READMD.md

연관 이슈

Pull Request 체크리스트

TODO

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

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 15, 2026

CLA assistant check
All committers have signed the CLA.

@gusdn3477 gusdn3477 changed the title [문서] C29 충돌 해결 및 미번역분 번역 [문서] C28 충돌 해결 및 미번역분 번역 May 15, 2026
@gusdn3477 gusdn3477 force-pushed the gusdn3477/sync-c28 branch from 17b649b to 17bc28b Compare May 15, 2026 13:37
@gusdn3477 gusdn3477 marked this pull request as ready for review May 16, 2026 12:32
@gusdn3477
Copy link
Copy Markdown
Author

@taejinchoi-cbnu
안녕하세요 태진님. 리뷰 부탁드려도 될까요?

@taejinchoi-cbnu
Copy link
Copy Markdown
Contributor

@taejinchoi-cbnu 안녕하세요 태진님. 리뷰 부탁드려도 될까요?

image

리뷰 진행하려고 했는데.. 지금 일감 배분 시트가 꼬여있어서 PR 올리신 파일들은 현재 시트 기준으로는 저랑 Turtle-Hwan님에게 할당 되어 있고 할당 부분이 공란으로 되어 있습니다.

image

디스코드에서 스레드로 말씀 나누실까요..? 조정이 필요해보입니다ㅠㅠ

Copy link
Copy Markdown
Contributor

@taejinchoi-cbnu taejinchoi-cbnu left a comment

Choose a reason for hiding this comment

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

세션 시작으로 일단 진행한 부분들 먼저 sumbit review 하겠습니다!

=======
In this example, the server code is not presented, as it's beyond our scope. The server accepts the POST request and replies "User saved".
>>>>>>> upstream/master
이 예시에서 요청받아 처리하는 서버 코드는 튜토리얼 범위를 벗어나므로 제시하지 않습니다. 서버는 POST 요청을 받고 "저장 성공"이라고 응답을 보내준다는 정도만 알고 계시면 됩니다.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"저장 성공" -> 'User data 저장 성공' 이런 느낌으로 가는건 어떤가요?

원문 "User saved"에 조금 더 가깝게 해석하는건 어떤가 하는 의견입니다!

Copy link
Copy Markdown
Contributor

@taejinchoi-cbnu taejinchoi-cbnu left a comment

Choose a reason for hiding this comment

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

모든 파일 검토 완료 했습니다!

- `PATCH` 메서드를 사용하고 있습니다.
- `Content-Type`이 `application/x-www-form-urlencoded`나 `multipart/form-data`, `text/plain` 아닙니다.
- `Content-Type`이 `application/x-www-form-urlencoded`나 `multipart/form-data`, `text/plain` 아닙니다.
- 비표준 헤더 `API-Key`를 사용합니다.
Copy link
Copy Markdown
Contributor

@taejinchoi-cbnu taejinchoi-cbnu May 17, 2026

Choose a reason for hiding this comment

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

비표준 헤더도 맞는 말로 알고 있는데 원문 "Unsafe"를 line 255에서는 안전하지 않은 헤더로 되어 있어서 통일이 필요해 보입니다!

=======
A call `socket.send(body)` allows `body` in string or a binary format, including `Blob`, `ArrayBuffer`, etc. No settings are required: just send it out in any format.
>>>>>>> upstream/master
`socket.send(body)`를 호출하면 `body`에는 문자열이나 `Blob`, `ArrayBuffer` 같은 이진 형식을 넣을 수 있습니다. 별도 설정 없이 원하는 형식으로 넣기만 하면 알아서 전송됩니다.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

etc.가 있어서 HEAD에 있는 "ArrayBuffer 등의" 문장 살리는건 어떤가요?
아니면 "~와 같은" 으로 Blob이나 ArrayBuffer 말고도 다른 것들을 body에 넣을 수 있다는 의미 살리는게 좋을 것 같습니다!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants