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 > 05-optimizing > 06-lazy-loading.mdx #368

Merged
merged 7 commits into from
Oct 9, 2023

Conversation

dev-sese
Copy link
Contributor

@dev-sese dev-sese commented Jun 30, 2023

기여자용

문서 개선

Progress

@devyuriii
Copy link
Contributor

ref #14


There are two ways you can implement lazy loading in Next.js:
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.

모범사례 참고

문서 전체적으로 반영 부탁드립니다 :)

Suggested change
Next.js에서 지연 로딩을 구현하는 데는 두 가지 방법이 있습니다:
Next.js에서 지연 로딩을 구현하는 데는 두 가지 방법이 있습니다.

@g2hhh2ee
Copy link
Contributor

g2hhh2ee commented Jul 20, 2023

수고하셨습니다! 리뷰 확인 부탁드립니다 🙂
Progress의 task도 체크해주세요 :)

@ChanghyeonYoon
Copy link
Member

@dev-sese 리뷰 이후 작업이 몇일 동안 이뤄지지 않아 9월 13일까지 작업이 없을 경우 이슈를 close하고 새로운 분에게 할당하겠습니다!

@dev-sese
Copy link
Contributor Author

dev-sese commented Sep 6, 2023

리뷰 사항을 이제야 확인했습니다. 리마인드 감사합니다!
Progress의 task도 체크 완료하였습니다.
감사합니다!

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.

안녕하세요 @dev-sese 님!
기여해주셔서 감사합니다 :)
추가적으로 번역 담당 챕터이셨던 02-app > 01-building-your-application > 05-optimizing > 06-lazy-loading.mdx를 제외하고, 나머지 PR에 함께 추적되어 올라온 4개의 파일은 제외해주시기 바랍니다!

@ChanghyeonYoon
Copy link
Member

@dev-sese
10/13 까지 충돌파일 해결 작업 진행 안될경우
이슈 클로즈하고 다른분에게 재할당 하겠습니다!

@dev-sese
Copy link
Contributor Author

dev-sese commented Oct 9, 2023

안녕하세요 기존에 담당이 아니지만 추적되어 함께 올라간 파일들을 작업 전으로 원복하였습니다.
제가 git 사용이 미숙해 여전히 에러가 있을 수 있습니다ㅠㅠ
확인해보시고 수정사항이 있다면 말씀해주세요!

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.

기여해주셔서 감사합니다.
추가적으로 할당 희망하시는 챕터의 경우 이슈에 작성 부탁드립니다!☺️

@yoo-jimin127 yoo-jimin127 merged commit a0ccd8b into Nextjs-kr:main Oct 9, 2023
@yoo-jimin127 yoo-jimin127 mentioned this pull request Oct 9, 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.

5 participants