Skip to content
This repository was archived by the owner on Mar 20, 2024. It is now read-only.

Trans(tutorial/toh-pt6.jade)#24

Merged
byungjune merged 9 commits intoangular:masterfrom
tiratano:master
Jan 13, 2017
Merged

Trans(tutorial/toh-pt6.jade)#24
byungjune merged 9 commits intoangular:masterfrom
tiratano:master

Conversation

@tiratano
Copy link
Contributor

  • Trans(tutorial/toh-pt6.jade) > 튜토리얼 6 번역
  • Trans(tutorial/toh-pt5.jade) > 띄어쓰기 수정 (또다른 => 또 다른)
  • Trans(tutorial/_data.json) > 튜토리얼 6 검토하려고 들어갔는데 제일 위쪽에 제목이 보이고 있습니다. 그런데 처음 보는 사람은 "우리의" 서비스 라는 것이 헷갈릴 수 있을 것 같아서 "우리의" 라는 표현을 삭제했습니다.

@tiratano
Copy link
Contributor Author

Trans(tutorial/toh-pt6.jade)가 반복되는 것은
에디터 차이로 끝에 공백이 있는 경우 remove & add로 분석이 되길래
공백문자를 최대한 맞춰서 add로만 맞추려고 한 것이니 무시하셔도 됩니다.


:marked
# 데이터 가져오기와 저장하기
# Getting and Saving Data
Copy link
Contributor

Choose a reason for hiding this comment

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

위의 부분 "#Getting and Saving Data"는 원문에서 삭제된 부분입니다.
원문과의 통일성을 위해 삭제하는 것이 좋을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

헉.. 문서가 업데이트 되면서 남아있었나보네요.
삭제하겠습니다.

Fortunately we're ready to import from `@angular/http` because `systemjs.config` configured *SystemJS* to load that library when we need it.

:marked
### Http 서비스 등록하기
Copy link
Contributor

Choose a reason for hiding this comment

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

Http를 대문자로 수정하는 것이 좋을 것 같습니다.

Http -> HTTP

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다.


We also adjust to the fact that the `data` in the response is a single hero object rather than !{_an} !{_array}.

### 변함없는 API
Copy link
Contributor

Choose a reason for hiding this comment

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

getHeroes 에 대한 해석이 누락되었습니다.
'변함없는 getHeroes API'로 수정하는 것이 좋을 것 같습니다 : )

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ㅠ 업데이트 변경분인가보네요. 수정하겠습니다.

+makeExcerpt('app/hero-detail.component.ts', 'save')

:marked
### 히어로 서비스의 업데이트 메소드
Copy link
Contributor

Choose a reason for hiding this comment

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

업데이트의 ``가 누락되었습니다.
596번째 라인의 delete는 영어로 표기되었는데 이곳은 한글로 표기되어 있습니다.
타 설명에서 `save`메소드 등도 영어로 표기하신 것 같아 `update` 메소드로 표기하는 것이 어떨까 싶습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

`` 수정했습니다.
지적하신 것처럼 해석에서는 한글로, 정확한 메소드 이름인 경우는 영어로 했는데..
여기만 한글로 했네요; 수정하겠습니다.

Copy link
Contributor Author

@tiratano tiratano left a comment

Choose a reason for hiding this comment

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

리뷰 내용 반영했습니다.

@tiratano
Copy link
Contributor Author

리뷰 감사합니다. 👍

@byungjune byungjune merged commit 9712f31 into angular:master Jan 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants