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

🐛 최초 접속 시 렌더링 문제 #26

Merged
merged 2 commits into from
Nov 1, 2023
Merged

🐛 최초 접속 시 렌더링 문제 #26

merged 2 commits into from
Nov 1, 2023

Conversation

oaoong
Copy link
Collaborator

@oaoong oaoong commented Nov 1, 2023

- 목적

관련 티켓 번호: 106


- 주요 변경 사항

  • ThemeProvider에서 마운팅에 따른 렌더링 제거

기타 사항 (선택)

  • 결과적으로 dev 모드에서의 해당 에러는 그대로 있습니다. production에서 문제가 없고, dev에서도 fatal하지 않기 때문에 남겨두었습니다.
  • 또한 차선책이 있기는 했는데, 시도해 본 바 production에서도 사소한 영향을 끼쳐 좋지 못하다 생각해 폐기했습니다.
  • 아마 원인은 nextJS dev에서만 적용되는 특성과 wrapper의 hook이 꼬여 문제가 생기는 것으로 추정됩니다.

- 스크린샷 (선택)

@oaoong oaoong added the D-0 label Nov 1, 2023
@oaoong oaoong self-assigned this Nov 1, 2023
@oaoong oaoong added the 작업 중 아직 작업 중이니 리뷰하지 말아주세요. label Nov 1, 2023
@oaoong oaoong removed the 작업 중 아직 작업 중이니 리뷰하지 말아주세요. label Nov 1, 2023
Copy link
Contributor

@juyeon-park juyeon-park left a comment

Choose a reason for hiding this comment

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

저도 제 컴퓨터 환경에서 확인해보니까 dev환경에서 어떨 땐 제대로 home이 나오고 어떨 땐 새로고침해야 나오네요. 제가 뜨는 오류에 대해서 stackoverflow들을 찾아보니까 encoding을 설치하라고 하는데 그래도 안된다는 사람들도 있다고 하고 프로젝트에 영향을 조금이라도 줄까봐 fatal하지 않으니 일단 개발 진행하겠습니다! 재희님 말씀대로 production에는 문제가 없기를 😢😢 추후에 production에 문제 생긴다면 날잡아서 회의해서 해결해봐요!

@oaoong oaoong merged commit 8dc20d4 into develop Nov 1, 2023
2 checks passed
@oaoong oaoong deleted the NABI-106 branch November 1, 2023 13:01
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.

2 participants