Skip to content

[async/await] article 번역 #345

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[async/await] article 번역 #345

wants to merge 1 commit into from

Conversation

sojeongw
Copy link
Contributor

@sojeongw sojeongw commented Oct 28, 2019 •
edited
Loading

Pull Request 체크리스트

TODO

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

  1. '함수 XX' 규칙이 이번 글에도 해당되는지 모르겠네요. 번역하면서 조금 어색하다고 느꼈거든요.
    특히 133, 254번 라인이 그랬습니다.

  2. 171번 라인 번역이 좀 헷갈리네요. 확인 부탁드리겠습니다.

@javascript-translate-bot javascript-translate-bot requested a review from a team October 28, 2019 01:37
@Violet-Bora-Lee
Copy link
Member

제가 메일로 공유드렸는데, 본 PR은 모든 request에서 503에러가 뜨네요...
깃헙 서버 에러같은데 티켓 만들어서 보내도 응답이 느리네요.

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Oct 29, 2019 •
edited
Loading

문의주신 사항에 대한 내용을 번역 모범사례에 등록하였습니다.

https://github.com/javascript-tutorial/ko.javascript.info/wiki/%EB%B2%88%EC%97%AD-%EB%AA%A8%EB%B2%94-%EC%82%AC%EB%A1%80#%ED%95%A8%EC%88%98

image

Violet-Bora-Lee added a commit that referenced this pull request Oct 29, 2019
@Violet-Bora-Lee
#345 [async와 await] 번역
66d65d3 
- co-author: @sojeongw
- promise.all관련 부분 추가 보완 필요함
- javascript-tutorial/en.javascript.info#1548 이슈 반영해야 함
@sojeongw
Copy link
Contributor Author

sojeongw commented Oct 30, 2019 •
edited
Loading

보완하신 내용 잘 보았습니다! 번역이 안된 부분은 제가 이어서 하는 건가요, 아니면 보라님이 나중에 해결하려고 남겨놓으신 건가요? 하실 예정이면 PR 닫으려구요!

@Violet-Bora-Lee
Copy link
Member

넵 제가 promise부분 마무리 하면서 같이 마무리하려구요.
pr은 클로즈해도 될것 같습니다
감사합니다 :)

@sojeongw sojeongw closed this Oct 30, 2019
@Violet-Bora-Lee Violet-Bora-Lee mentioned this pull request Nov 8, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
review needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sojeongw @Violet-Bora-Lee @javascript-translate-bot