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

[PR] 전체 마무리 및 댓글 기능 구현 #61

Merged
merged 3 commits into from
May 17, 2024
Merged

Conversation

kangsinbeom
Copy link
Contributor

📋 Checklist

  • 🔀 PR 제목의 형식을 잘 작성했나요? (e.g. feat: 유저 조회 기능 구현)
  • 🏷️ 라벨, 프로젝트는 등록했나요?
  • 🧹 코드 스멜은 해결했나요?

🧩 이슈 번호


🎨 완성 이미지

작업 이미지

👩‍💻 공유 포인트 및 논의 사항

  • 댓글기능 구현 예정 무조건 금요일에 완료할 예정!
  • �api 수정 에러 바운더리 생성

401과 400에 대해서 에러 메세지가 알럿창으로 뜨게끔 구현하였음
@kangsinbeom kangsinbeom added the enhancement New feature or request label May 17, 2024
@kangsinbeom kangsinbeom self-assigned this May 17, 2024
@kangsinbeom kangsinbeom linked an issue May 17, 2024 that may be closed by this pull request
<Suspense fallback={<>loading</>}>
<BrowserRouter>
<ErrorBoundary navigate={navigate}>
<Suspense fallback={<>loading</>}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

suspense 이거 삭제해주세요.
메인 페이지 경우에는 Nav, MainList, MainExtra가 MainList 컴포넌트만 독립적으로 분리 시킬거라서 suspense 해놨습니다.

@@ -13,48 +13,48 @@ import ErrorPage from '../pages/error';
export const routers = [
{
path: '/',
component: MainPage,
element: MainPage,
Copy link
Collaborator

Choose a reason for hiding this comment

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

component -> element 변경하신 이유를 알고싶군요.

@kangsinbeom kangsinbeom merged commit 173f93c into dev May 17, 2024
@kangsinbeom kangsinbeom deleted the develop/comment branch May 17, 2024 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[develop/comment] 전체 마무리 및 댓글 기능 구현
3 participants