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.

Sign up for GitHub

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

Jump to bottom

Refactoring : 불필요한 layout파일 제거 #87

Merged
merged 4 commits into from
Jul 8, 2023
Merged

Refactoring : 불필요한 layout파일 제거 #87

merged 4 commits into from
Jul 8, 2023

Conversation

guesung
Copy link
Member

@guesung guesung commented Jul 7, 2023

💡 왜 PR을 올렸나요?

💁 무엇이 어떻게 바뀌나요?

  1. 불필요한 layout을 제거하였습니다.
  • 기존에 layout.tsx를 제거하면 발생하던 오류는, 빌드 후 해당 페이지에 접속 시 .next/static/chunks/app/join/step1폴더에 각 페이지를 정적으로 만들어 캐싱을 해둡니다. layout.tsx가 있는 페이지에 접속 시, 정적으로 LayoutGroup.js를 저장해둡니다. 그런데 이 파일을 제거하고 다시 그 페이지로 이동 시 그 layout.tsx를 찾는데 없어서 에러가 발생한 것이었습니다.
  • 따라서, Next.js 후 yarn dev를 다시 실행하면 정상적으로 동작합니다.
스크린샷 2023-07-07 오후 4 39 40

💬 리뷰어분들께

@github-actions github-actions bot added this to the 7월 1주차 Sprint milestone Jul 7, 2023
kangju2000
kangju2000 approved these changes Jul 8, 2023
View reviewed changes
Copy link
Contributor

@kangju2000 kangju2000 left a comment

Choose a reason for hiding this comment

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

해결하셨군요👍

@guesung guesung merged commit 4d54358 into develop Jul 8, 2023
1 check passed
@guesung guesung deleted the feature/refactoring-delete-unused-layout branch July 8, 2023 09:56
@guesung guesung added the Refactoring 리팩토링을 수행한 경우 label Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kangju2000 kangju2000 kangju2000 approved these changes

Assignees

@guesung guesung

Labels
Refactoring 리팩토링을 수행한 경우
Projects
None yet
Milestone
7월 1주차 Sprint
Development

Successfully merging this pull request may close these issues.

[BUG SOLVE] layout.tsx가 없으면 발생하는 에러
2 participants
@guesung @kangju2000