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

[FE] 리뷰 모아보기 페이지 API 로직 연동 #817

Merged
merged 10 commits into from
Oct 10, 2024

Conversation

chysis
Copy link
Contributor

@chysis chysis commented Oct 10, 2024


🚀 어떤 기능을 구현했나요 ?

  • 리뷰 모아보기 페이지의 API 로직을 연동했습니다.

🔥 어떻게 해결했나요 ?

  1. 섹션 리스트를 받아옵니다.
  2. 드롭다운에서 섹션을 선택할 때마다 해당하는 섹션의 답변 데이터를 받아옵니다.

섹션 리스트의 경우 전체 섹션 리스트를 받아오기 때문에 변경 사항이 적을 것이라 판단, staleTime을 1시간으로 설정했습니다.
섹션별 답변 데이터의 경우 기존 리뷰 목록 데이터와 마찬가지로 staleTime을 1분으로 설정했습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 테스트는 추후 작성하도록 하겠습니다

📚 참고 자료, 할 말

Copy link
Contributor

@BadaHertz52 BadaHertz52 left a comment

Choose a reason for hiding this comment

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

꾿!!

@chysis
Copy link
Contributor Author

chysis commented Oct 10, 2024

추후 반영할 것

테스트 작성

  • api 요청 테스트

에러 처리

  • 현재 섹션 리스트, 섹션별 리뷰를 불러오는 것에 실패하면 FallbackUI 제공
    • 해결 방법 모색

Copy link
Contributor

@ImxYJL ImxYJL left a comment

Choose a reason for hiding this comment

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

고생했어요~~~ 런칭에 가까워졌다!!

Copy link
Contributor

@soosoo22 soosoo22 left a comment

Choose a reason for hiding this comment

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

api 로직 연동하느라 수고했어요 에프이!

@chysis chysis merged commit 046b230 into develop Oct 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
4 participants