🎉 제안 요청, 카드 삭제 api 연결 완료 #91

juyeon-park · 2023-11-21T12:41:34Z

- 목적

- 주요 변경 사항

카드 삭제 api 연결완료
제안 요청 api 연결완료
상세 정보 페이지에서 내 물건일 경우 더보기 버튼 안나오던 버그 수정
상세 정보 페이지에서 내 물건일 경우 제안확인 버튼을 누르면 제안확인 페이지로 이동하도록 수정했습니다.
내 물건이 아닐때만 제안가능한 카드를 가져오도록 useSuggestionsQuery에 enabled 속성 추가

기타 사항 (선택)

동현님께서 지난번에 suggest-> suggestion으로 통일하자고 해주신 것 반영해서 파일명 useSuggestMutation.ts -> useSuggestionUpdateMutation.ts로 변경했습니다.
이번엔 리뷰 답변 pending 상태인지 확인하고 start review 꼭 누르겠습니다 🥲

- 스크린샷 (선택)

보고있는 상품에 제안할 수 있는 내 카드가 없을 경우

보고있는 상품에 제안할 수 있는 내 카드가 있을 경우

오퍼하기를 눌렀을 때의 ui

…ient into NABI-238

doggopawer · 2023-11-21T13:18:46Z

src/app/(root)/(routes)/cards/[cardId]/components/trade-section/TradeSection.tsx

@@ -77,13 +77,12 @@ const TradeSection = ({

  const onClickButton = async () => {
    if (isMyItem) {
-      alert('제안확인 페이지로 이동하기')
+      router.push(`/mypage/suggestions/${cardId}`)


이번 제 PR에 AppPath.mySuggestions 올려두었습니다! 차후에 적용하시면 될 것 같습니다.

오 AppPath에 있었군요! 바로 수정하겠습니다!

appPath파일을 보니 내 제안확인 페이지의 루트가 이렇게 설정되어 있더라고요!

mySuggestions: () => '/mypage/suggestions` as const,

카드 아이디가 슬러그 형태여서 매개변수에 cardId를 받게 이렇게 고쳤습니다! 괜찮을까요? MyCard.tsx의 링크 주소도 수정해놨습니다!

mySuggestions: (cardId: number) => `/mypage/suggestions/${cardId}` as const

doggopawer · 2023-11-21T13:21:46Z

src/components/domain/card/suggest-card/SuggestCard.tsx

@@ -30,7 +30,7 @@ const SuggestCard = ({
  toCardId,
  suggestionStatus,
 }: SuggestCardProps) => {
-  const { mutate } = useSuggestMutation(toCardId, fromCardId)
+  const { mutate } = useSuggestionUpdateMutation(toCardId, fromCardId)


해당 쿼리는 post API를 호출하고 있으므로, update 보단 Create 가 더 어울리는 이름인 것 같습니다!

오 좋은 의견 감사합니다. post니까 create나 post가 더 좋겠네요 👍

doggopawer · 2023-11-21T13:22:41Z

src/hooks/api/mutations/useSuggestionUpdateMutation.ts

        const updateSuggestions: any = structuredClone(previousSuggestions)
-        console.log(updateSuggestions)
        //낙관적 업데이트
        const indexToUpdate = updateSuggestions.findIndex(
          (card: AvailableCardSuggestionListRes) =>


저번에 안됐던 문제가 혹시 structuredClone 을 안써서 그런게 맞나요?

넵 맞습니다! 깊은 복사를 하지 않아서 생긴 문제였습니다

doggopawer · 2023-11-21T13:23:24Z

src/services/suggestion/suggestion.ts

+    {},
+    {
+      'Content-Type': 'application/json',
+    },


덕분에 안 헤매고, 제 함수에도 적용했습니다.

…ient into NABI-238

juyeon-park added 6 commits November 21, 2023 16:28

🚧 제안 요청 api 연결 완료

edf95d0

Merge branch 'develop' of https://github.com/team-nabi/nabi-market-cl…

af0093f

…ient into NABI-238

🐛 내 물건일때 더보기 버튼 안나오는 버그 수정

51dd9ca

🎉 제안 요청, 카드 삭제 api 연결 완료

c6eda64

🧹 파일 이름 변경

d1e2e18

🧹 tanstack query devtool 안보이게 복구

dbfdccb

juyeon-park added the D-0 label Nov 21, 2023

juyeon-park requested review from oaoong and doggopawer November 21, 2023 12:41

juyeon-park self-assigned this Nov 21, 2023

doggopawer approved these changes Nov 21, 2023

View reviewed changes

oaoong approved these changes Nov 21, 2023

View reviewed changes

juyeon-park added 2 commits November 22, 2023 13:16

Merge branch 'develop' of https://github.com/team-nabi/nabi-market-cl…

1d081b8

…ient into NABI-238

✨ 코드리뷰 반영

119a456

juyeon-park merged commit 776fbdd into develop Nov 22, 2023
2 checks passed

juyeon-park deleted the NABI-238 branch November 22, 2023 05:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🎉 제안 요청, 카드 삭제 api 연결 완료 #91

🎉 제안 요청, 카드 삭제 api 연결 완료 #91

juyeon-park commented Nov 21, 2023

doggopawer Nov 21, 2023

juyeon-park Nov 22, 2023

juyeon-park Nov 22, 2023 •

edited

Loading

doggopawer Nov 21, 2023

juyeon-park Nov 22, 2023

doggopawer Nov 21, 2023

juyeon-park Nov 22, 2023

doggopawer Nov 21, 2023

juyeon-park Nov 22, 2023

🎉 제안 요청, 카드 삭제 api 연결 완료 #91

🎉 제안 요청, 카드 삭제 api 연결 완료 #91

Conversation

juyeon-park commented Nov 21, 2023

- 목적

- 주요 변경 사항

기타 사항 (선택)

- 스크린샷 (선택)

Choose a reason for hiding this comment

Choose a reason for hiding this comment

juyeon-park Nov 22, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

juyeon-park Nov 22, 2023 •

edited

Loading