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

Trans (guide 9)#30

Merged
byungjune merged 3 commits intoangular:masterfrom
tiratano:master
Jan 20, 2017
Merged

Trans (guide 9)#30
byungjune merged 3 commits intoangular:masterfrom
tiratano:master

Conversation

@tiratano
Copy link
Contributor

@tiratano tiratano commented Jan 19, 2017

  • Trans (guide 9)
  • Fix: tutorial/toh-pt3.jade 메타데이타 => 메타데이터

Copy link
Contributor

@byungjune byungjune left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 : )

+makeExample('dependency-injection/ts/app/providers.component.ts','providers-6a')(format=".")

:marked
해법은: useExisting 옵션을 사용한 별칭입니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

useExisting의 ``가 누락되었습니다 : )

+makeExample('dependency-injection/ts/app/heroes/hero.service.ts','internals', 'app/heroes/hero.service.ts (excerpt)')(format='.')

:marked
`Logger`를 주입할 수 있지만, 그러나 boolean `isAuthorized`를 주입할 수는 없습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

중간의 but에 대한 해석이 2번 들어간 것 같습니다.

중간의 그러나를 제거하여
[Logger를 주입할 수 있지만, 그러나 boolean isAuthorized를 주입할 수는 없습니다.]를
[Logger를 주입할 수 있지만, boolean isAuthorized를 주입할 수는 없습니다.]로 하면 좀 더 자연스러울 것 같습니다 : )


.l-sub-section
:marked
`Car` 소비자는 문제가 있습니다. 소비자는 다음과 같이
Copy link
Contributor

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.

리뷰 사항 수정했습니다. : )

Copy link
Contributor

@byungjune byungjune left a comment

Choose a reason for hiding this comment

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

확인했습니다. 수고하셨습니다 : )

@byungjune byungjune merged commit 898b329 into angular:master Jan 20, 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