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

feat(main): main page 작업 #153

Merged
merged 18 commits into from
Oct 9, 2023

Conversation

SeolJaeHyeok
Copy link
Collaborator

📌 이슈 링크

  • close #

📖 작업 배경

  • 프로젝트 내의 apollo 버전이 너무 낮아 현재 메이저 버전으로 맞춰주기 위해 작업을 진행했습니다.

🛠️ 구현 내용

  • 기존 apollo server(v3)를 apollo server(v4) + express로 migration 했습니다.
  • 메인 페이지의 기능(리스트 불러오기, 페이지네이션, 태그 필터링)을 작업했습니다.
  • 런타임 에러 처리를 위해 Error Boundary 적용

💡 참고사항

  • Suspense를 적용했다가 생각해보니 server fetching의 경우 suspense가 필요없을 것 같아 지워줬습니다.
  • next12 -> next 13으로 넘어오면서 next/router 가 아닌 next/navigation 모듈을 사용하지 않으면 에러를 뱉어줍니다.
    기존 useRouter Hook이 하는 일이 너무 많다고 생각해서 당연히 분리되어야 된다고 생각했는데 하나의 Hook에서 했던 작업을 usePathname, useSearchParams, useRouter로 분리해서 작성하려다 보니 코드량이 늘어나고 같은 코드를 반복하는 경우가 많아 useCustomNavigate 라는 Hook을 만들어 로직을 한 공간에서 관리하려고 했습니다.
    근데 작성하면서도 best practice는 아닌거 같다는 생각이 들어 다른 분들은 next13에서 navigation을 어떻게 우아하게 처리하시는지 궁굼합니다!!

🖼️ 스크린샷


SeolJaeHyeok and others added 11 commits September 22, 2023 22:04
1. 원래는 Pagination 컴포넌트를 패키지로 빼서 의존시켜 사용할 예정이었으나 패키지를 빌드 시켰을 때 스타일이 적용되지 않는 문제가 발생하여 일단은 같은 프로젝트 내에서 컴포넌트 생성하여 작업 -> 추후에는 결국 분리해야한다.
2. 스타일 보다는 기능에 집중하여 코드 작업
아티클 상세 페이지 라우팅까지 완료
@SeolJaeHyeok SeolJaeHyeok changed the title feature: main page feat(main): main page 작업 Oct 4, 2023
@SeolJaeHyeok SeolJaeHyeok merged commit 7060f8a into pagers-org:team2/SeolJaeHyeok Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant