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

NEXT 버전 업데이트에 따른 page_router 에서 app_router로 이전 #149

Merged
merged 24 commits into from
Jan 19, 2024

Conversation

bae-sh
Copy link
Member

@bae-sh bae-sh commented Jan 15, 2024

DONE

  • page router -> app router로 이전 완료 (13.2 -> 14.0)
  • 게시글 서버컴포넌트 seo 최적화 구현
  • 홈 화면의 피드 게시글들 html 주입 하는 방식을 xss 방어를 위해 dompurify 라이브러리 설치

TODO

  • 현재 홈 화면의 게시글을 html 주입으로 인해 hydration 에러가 발생
    -> 이를 제거하기 위해서는 게시글을 서버사이드가 아닌 클라이언트 사이드에서 렌더링을 해야하는데 그렇게 되면 유저 경험이 낮아짐.
    디자인 변경 시 미리보기 요약글을 유저로부터 받을 예정(벨로그방식)이므로 이는 string 값이라 주입할 필요 없어 문제가 해결 될것이라 생각

혹시 Approuter에 대해서 디코를 통해 전반적인 흐름을 알려줄테니 일정을 잡아보자!!

@bae-sh bae-sh self-assigned this Jan 15, 2024
@bae-sh bae-sh added Feature 기능 구현 Refactor 리팩토링 labels Jan 15, 2024
Copy link

⚡️ Lighthouse report 0

Category Score
🟢 Performance 98
🟢 Accessibility 97
🟢 Best practices 93
🟢 SEO 100
🔴 PWA 22

⚡️ Lighthouse report 1

Category Score
🟢 Performance 99
🟢 Accessibility 97
🟢 Best practices 93
🟢 SEO 100
🔴 PWA 22

⚡️ Lighthouse report 2

Category Score
🟢 Performance 99
🟢 Accessibility 97
🟢 Best practices 93
🟢 SEO 100
🔴 PWA 22

@bae-sh bae-sh merged commit 680b5e1 into dev Jan 19, 2024
2 checks passed
@bae-sh bae-sh deleted the feat/#142-app-router-migration branch January 19, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature 기능 구현 Refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants