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 > 03-next-config-js > env.mdx #346

Closed
wants to merge 1 commit into from
Closed

Conversation

ssseok
Copy link
Contributor

@ssseok ssseok commented Jun 28, 2023

기여자용

문서 개선

Progress

@ssseok
Copy link
Contributor Author

ssseok commented Jun 29, 2023

🙌피드백 부탁드립니다.🙌

@ChanghyeonYoon ChanghyeonYoon added the documentation 문서 작업 label Jun 29, 2023
@haileyport haileyport linked an issue Jun 30, 2023 that may be closed by this pull request
@chaejunlee chaejunlee self-requested a review July 11, 2023 06:00
Copy link
Contributor

@chaejunlee chaejunlee left a comment

Choose a reason for hiding this comment

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

번역에 기여해주셔서 감사합니다! 수정 사항 커멘트 남겨드렸으니 확인 부탁드립니다!


</details>

> **Good to know**: environment variables specified in this way will **always** be included in the JavaScript bundle, prefixing the environment variable name with `NEXT_PUBLIC_` only has an effect when specifying them [through the environment or .env files](/docs/pages/building-your-application/configuring/environment-variables).
> **알아두면 좋은 사실**. 이런 방식으로 지정된 환경 변수들은 **항상** 자바스크립트 번들에 포함됩니다. 환경 변수 이름 앞에 `NEXT_PUBLIC_`를 붙이는 것은 그것들을 [환경 또는 .env 파일을 통해](/docs/pages/building-your-application/configuring/environment-variables) 지정할 때만 효과가 있습니다.
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
> **알아두면 좋은 사실**. 이런 방식으로 지정된 환경 변수들은 **항상** 자바스크립트 번들에 포함됩니다. 환경 변수 이름 앞에 `NEXT_PUBLIC_`를 붙이는 것은 그것들을 [환경 또는 .env 파일을 통해](/docs/pages/building-your-application/configuring/environment-variables) 지정할 때만 효과가 있습니다.
> **참고**: 이런 방식으로 지정된 환경 변수들은 **항상** 자바스크립트 번들에 포함됩니다. 환경 변수 이름 앞에 `NEXT_PUBLIC_`를 붙이는 것은 환경 변수들을 [환경 또는 .env 파일을 통해](/docs/pages/building-your-application/configuring/environment-variables) 지정할 때만 효과가 있습니다.


For example, the following line:
예를 들어 다음과 같은 줄.
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
예를 들어 다음과 같은 줄.
예를 들어 다음과 같은 줄은,


```jsx
return <h1>The value of customKey is: {process.env.customKey}</h1>
```

Will end up being:
다음과 같이 변환될 것입니다.
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
다음과 같이 변환될 것입니다.
아래와 같이 변환될 것입니다.

@@ -34,15 +34,15 @@ function Page() {
export default Page
```

Next.js will replace `process.env.customKey` with `'my-value'` at build time. Trying to destructure `process.env` variables won't work due to the nature of webpack [DefinePlugin](https://webpack.js.org/plugins/define-plugin/).
Next.js는 빌드 시에 `process.env.customKey``'my-value'`로 대체할 것입니다. webpack [DefinePlugin]https://webpack.js.org/plugins/define-plugin/)의 특성 때문에 `process.env` 변수를 구조 분해하는 것은 작동하지 않습니다.
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
Next.js는 빌드 시에 `process.env.customKey``'my-value'`로 대체할 것입니다. webpack [DefinePlugin]https://webpack.js.org/plugins/define-plugin/)의 특성 때문에 `process.env` 변수를 구조 분해하는 것은 작동하지 않습니다.
Next.js는 빌드 시에 `process.env.customKey``'my-value'`로 대체할 것입니다. 웹팩 [DefinePlugin](https://webpack.js.org/plugins/define-plugin/)의 특성 때문에 `process.env` 변수는 구조 분해할 수 없습니다.

@chaejunlee
Copy link
Contributor

chaejunlee commented Jul 11, 2023

안녕하세요! 좋은 기회 주셔서 감사합니다.

03-pages > 03-next-config-js > 03-next-config-js > env.mdx 번역 신청합니다.

#1 (comment)

신청은 페이지 디렉토리로 해주셨지만 번역은 앱 디렉토리로 해주셨습니다..ㅠ

페이지 디렉토리로 다시 번역해주실 수 있을까요?

file path는 docs/03-pages/02-api-reference/03-next-config-js/env.mdx 입니다.

@chaejunlee chaejunlee added invalid 할당받은 티켓과 다른 작업 duplicate App, Pages 공통 문서 labels Jul 11, 2023
@ssseok
Copy link
Contributor Author

ssseok commented Jul 11, 2023

헉... 이른 시일에,곧 번역하겠습니다. 잘못 번역을 하고 있었군요...

@ssseok
Copy link
Contributor Author

ssseok commented Jul 11, 2023

다시 fork 해서 다시 올려도 괜찮을까요?

@ssseok ssseok changed the title docs: 02-app > 02-api-reference > 05-next-config-js > env.mdx docs: 03-pages > 02-api-reference > 03-next-config-js > env.mdx Jul 11, 2023
@ssseok ssseok closed this by deleting the head repository Jul 11, 2023
@ssseok
Copy link
Contributor Author

ssseok commented Jul 11, 2023

수정해서 다시 pr 올리겠습니다 죄송합니다.

@chaejunlee
Copy link
Contributor

@ssseok 아닙니다! 천천히 작업하시고 pr 올려주세용

@ssseok
Copy link
Contributor Author

ssseok commented Jul 11, 2023

@chaejunlee #418 다시 수정하여 수정본 올렸습니다! 확인 부탁드립니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 문서 작업 duplicate App, Pages 공통 문서 In Review invalid 할당받은 티켓과 다른 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

env.mdx
3 participants