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

03-pages > 01-building-your-application > 03-data-fetching > 02-get-static-paths.mdx #430

Merged
merged 3 commits into from
Sep 5, 2023

Conversation

guesung
Copy link
Contributor

@guesung guesung commented Aug 13, 2023

기여자용

문서 개선

Progress

@github-actions github-actions bot added the documentation 문서 작업 label Aug 13, 2023
@guesung guesung changed the title translate : getStaticPaths 번역 03-pages > 01-building-your-application > 03-data-fetching > 02-get-static-paths.mdx Aug 13, 2023
@YuriKwon YuriKwon self-requested a review August 22, 2023 12:47
Copy link
Contributor

@YuriKwon YuriKwon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 리뷰 확인 및 반영 부탁드립니다:)


You should use `getStaticPaths` if you’re statically pre-rendering pages that use dynamic routes and:
- 데이터가 무표시 CMS에서 오는 경우
Copy link
Contributor

Choose a reason for hiding this comment

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

headless CMS를 헤드리스 CMS로 수정하면 어떨까요??


`getStaticPaths` allows you to control which pages are generated during the build instead of on-demand with [`fallback`](/docs/pages/api-reference/functions/get-static-paths#fallback-blocking). Generating more pages during a build will cause slower builds.
`paths`에 대한 빈 배열을 반환함으로써 모든 페이지를 요청 시 생성을 연기할 수 있습니다. 이는 Next.js 애플리케이션을 여러 환경에 배포할 때 특히 유용합니다. 예를 들어, 미리보기에는 모든 페이지를 요청 시 생성하여 빌드 속도를 높일 수 있지만(프로덕션 빌드는 제외) 수백/수천 개의 정적 페이지가 있는 사이트에 유용합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

문장이 조금 자연스럽지 않은 것 같습니다! 어순 변경 부탁드립니다:)

@guesung
Copy link
Contributor Author

guesung commented Aug 27, 2023

@YuriKwon

주신 피드백 반영해서 커밋 올려두었습니다.

꼼꼼한 피드백 감사드립니다!

@guesung guesung requested a review from YuriKwon August 27, 2023 13:47
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 087f8f0 into Nextjs-kr:main Sep 5, 2023
@yoo-jimin127
Copy link
Contributor

ref #60

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.

None yet

3 participants