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

docs: 03-pages > 02-api-reference > 02-functions > get-static-props.mdx #333

Merged
merged 8 commits into from
Aug 5, 2023

Conversation

NamJwong
Copy link
Contributor

@NamJwong NamJwong commented Jun 28, 2023

기여자용

문서 개선

Progress

closes: #105

@haileyport
Copy link
Contributor

ref #105

@haileyport haileyport requested review from haileyport and removed request for ChanghyeonYoon July 7, 2023 01:02
Copy link
Contributor

@haileyport haileyport left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 리뷰 반영 부탁드립니다~

@@ -1,9 +1,9 @@
---
title: getStaticProps
description: API reference for `getStaticProps`. Learn how to use `getStaticProps` to generate static pages with Next.js.
description: `getStaticProps`를 위한 API 레퍼런스입니다. 'getStaticProps'를 사용하여 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.

두 번째 getStaticProps를 백틱으로 감싸주세요!

@@ -40,34 +40,34 @@ export default function Page({ repo }) {
}
```

You can import modules in top-level scope for use in `getStaticProps`. Imports used will **not be bundled for the client-side**. This means you can write **server-side code directly in `getStaticProps`**, including fetching data from your database.
`getStaticProps`에서 사용할 모듈을 최상위 스코프로 가져올 수 있습니다. 사용된 가져오기는 **클라이언트 측에서 번들링 되지 않습니다.** 즉, 데이터베이스에서 데이터를 가져오는 것을 포함하여 **서버측 코드를 'getStaticProps'에 직접 작성할 수 있습니다.**
Copy link
Contributor

Choose a reason for hiding this comment

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

두 번째로 나오는 getStaticProps을 백틱으로 감싸주세요!

| `locale` | Contains the active locale (if enabled). |
| `locales` | Contains all supported locales (if enabled). |
| `defaultLocale` | Contains the configured default locale (if enabled). |
| Name | Description |
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

@haileyport haileyport 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
Copy link
Contributor

@NamJwong 님 안녕하세요!
Next.js kr 리뷰 반영 확인 및 머지 담당자입니다!
번역 진행해주신 해당 PR에서 충돌이 발생하고 있습니다.
최종 번역된 내용 바탕으로 해당 부분 충돌 해결 완료해주신 뒤,
저를 멘션해주시면 바로 리뷰 진행한 뒤 머지하도록 하겠습니다!
감사합니다 :)

@NamJwong
Copy link
Contributor Author

NamJwong commented Aug 2, 2023

@yoo-jimin127 충돌 해결했습니다 확인 부탁드립니다 :)

@NamJwong NamJwong changed the title docs: [WIP] 03-pages > 02-api-reference > 02-functions > get-static-props.mdx docs: 03-pages > 02-api-reference > 02-functions > get-static-props.mdx Aug 2, 2023
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.

@NamJwong 님 안녕하세요!
충돌 해결해주신 것 확인하였습니다 :)
감사합니다!

머지를 진행하려 하였으나,
충돌 해결 과정 중에서 번역이 완료되어 머지된 문서들이 영문으로 원상복구되어 함께 스테이징되어 푸시된 것으로 확인됩니다 😂

@NamJwong 님께서 번역 진행해주신 파일인 get-static-props.mdx를 제외하고 기존 파일을은 files changed에서 제외해주시기 바랍니다!
감사합니다 :)

@NamJwong
Copy link
Contributor Author

NamJwong commented Aug 4, 2023

@yoo-jimin127 처음 충돌 해결 시 있었던 실수를 되돌려서 다시 올바르게 충돌 해결했습니다 번거롭게 해드려 죄송합니다😂

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 7997f4f into Nextjs-kr:main Aug 5, 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.

get-static-props.mdx
4 participants