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] 상품 정보 조회 api #98

Merged
merged 8 commits into from
Nov 29, 2024
Merged

Conversation

seong-hui
Copy link
Collaborator

관련 이슈 🛰️

해결한 이슈 번호를 작성해주세요
close #96

작업 내용 🧑‍💻

작업한 내용을 간략히 작성해주세요

  • 기존 영경이의 getProduct를 getRelatedProduct로 변경하고 제가 getProduct를 먹었습니다. ㅋ.ㅋ
  • 상품 정보를 조회하는 api를 구현하였습니다.

PR 포인트 🗯️

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • useProductDetail의 useQuery에서 initialData를 더미데이터로 지정하여 서버 요청 실패 시에도 더미 데이터로 유지될 수 있도록 하였습니다.
  • 아직 별점이랑 리뷰수는 데이터 반영을 하지 못햇습니다. 왜냐하면 별점 버튼을 이전 api pr에서 반영햇기 떄문입니다 ,,ㅎㅎ 그래서 그 데이터는 머지하고 반영할게요!
  • 모두 responseType을 axios내에서 정의하셔서 저도 기존의 type 삭제했습니다
  • 궁금한 부분 질문 남겨주세요!

알게된 점 🚀

기록하며 개발하기!

  • useQuery의 옵션중에 placeholderData와 initialData가 비슷해보여서 이에 대해서 찾아보니
    • 서버가 끊겼을 때 사용자가 알아야 한다면 placeholderData가 적합.
    • 사용자가 끊김을 알아차리지 못하도록 기본 데이터를 계속 보여주고 싶다면 initialData가 적합.
  • 라고 합니다! 그래서 저는 initialData를 선택했습니당.

참고 자료 📖 (선택)

참고했던 문서들 공유하기!

스크린샷 📸 (선택)

Nov-29-2024 05-00-42

@seong-hui seong-hui self-assigned this Nov 28, 2024
@bykbyk0401
Copy link
Collaborator

placeholderData와 initialData 차이가 있는줄은 몰랐네요!!!
덕분에 차이 알아갑니당ㅎㅎ
충돌만 해결하면 될 것 같아요

@seong-hui seong-hui merged commit 5fa5074 into develop Nov 29, 2024
2 checks passed
@seong-hui seong-hui deleted the feat/#96/product-info-api branch November 29, 2024 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 상품 정보 조회 api
2 participants