-
Notifications
You must be signed in to change notification settings - Fork 202
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
Translate : docs > 02-app > 02-api-reference > 05-next-config-js > poweredByHeader.mdx #365
base: main
Are you sure you want to change the base?
Conversation
@ChanghyeonYoon 해당 건 작업 완료했는데 이슈가 있을까요? Progress Task 처리가 안되는거 같아서요. |
@fomerna 체크박스가 하나도 안돠어있어서요!! 체크박스 |
@ChanghyeonYoon 아 체크박스는 제가 하면 되는건가요?? 리뷰 전 체크박스까지 완료했습니다! |
@fomerna 저희가 pr이 많이 쌓여서 오래된거부터 순차적으로 진행중이라 순서되시면 리뷰남기고 멘션드리겠습니다! |
@@ -1,9 +1,9 @@ | |||
--- | |||
title: poweredByHeader | |||
description: Next.js will add the `x-powered-by` header by default. Learn to opt-out of it here. | |||
title: x-powered-by 헤더 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
원문과 다르게 x-powered-by 헤더
로 번역하신 이유가 있을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chaejunlee 의역을 했는데 원본 그대로 poweredByHeader 그대로 적어야할까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저의 확인이 늦었네요!! 죄송합니다!! 해당 문서는 config 값에 대한 서술을 하고 있으므로 원문, poweredByHeader, 그대로 남기는 것이 맞아 보입니다. 혹시 늦었지만 수정 가능하실까요?
@fomerna 리뷰 이후 작업이 몇일 동안 이뤄지지 않아 9월 13일까지 작업이 없을 경우 이슈를 close하고 새로운 분에게 할당하겠습니다! |
기여자용
문서 개선
pnpm prettier-fix
를 실행하여 서식 문제를 해결합니다. - 문서 기여 가이드를 읽고 문서 지침을 따르는지 확인하세요: https://github.com/Nextjs-kr/Nextjs.ko/blob/main/packages/next/README.mdProgress
pnpm prettier-fix