Skip to content
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.

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

02-app > 01-building-your-application > 03-data-fetching > 01-fetching.mdx #422

Merged
merged 2 commits into from
Aug 31, 2023

Conversation

2taesung
Copy link

@2taesung 2taesung commented Jul 15, 2023

기여자용

문서 개선

Progress

@haileyport haileyport linked an issue Jul 18, 2023 that may be closed by this pull request
@2taesung 2taesung removed their assignment Jul 20, 2023
@2taesung
Copy link
Author

리뷰 부탁드립니다 🕵️‍♂️


- [`cookies()`](/docs/app/api-reference/functions/cookies)
- [`headers()`](/docs/app/api-reference/functions/headers)

## `use` in Client Components
## 클라이언트 컴포넌트들에서 `use`
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## 클라이언트 컴포넌트들에서 `use`
## 클라이언트 컴포넌트들에서의 `use`

Copy link
Author

@2taesung 2taesung Aug 2, 2023

Choose a reason for hiding this comment

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

이 부분은 '에서의' 보다 '에서' 가 좀 더 자연스럽지 않을까요 ????
의미는 둘다 같다고 생각되고 '에서의'가 더 번역체라고 생각합니다!

@sungh0lim sungh0lim added the documentation 문서 작업 label Jul 31, 2023
@2taesung
Copy link
Author

2taesung commented Aug 2, 2023

@HA-SEUNG-JEONG
언급해주신 리뷰 '에서' -> '에서의' 를 제외하고 동의하여 수정해 pr 올릴 예정입니다!
'에서' -> '에서의' 의견 부탁드립니다!!

감사합니다!!

@2taesung
Copy link
Author

리뷰 반영했습니다!
확인 부탁드립니다~~~~!

Copy link
Contributor

@yoo-jimin127 yoo-jimin127 left a comment

Choose a reason for hiding this comment

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

LGTM!! 기여해주셔서 감사합니다 :) 👍
추가적으로 번역 희망하시는 챕터의 경우 이슈에 작성 부탁드려요!

@yoo-jimin127 yoo-jimin127 merged commit 43a8087 into Nextjs-kr:main Aug 31, 2023
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.

01-fetching.mdx
5 participants