[GGFE-215] 상점 api 연결 - 뽑기 모달, 상점 아이템 #971
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yoouyeon
reviewed
Sep 5, 2023
Comment on lines
35
to
50
| const res = await instance.patch('/pingpong/users/background', data); | ||
| if (res.data) { | ||
| setModal({ | ||
| modalName: 'USE-ITEM-GACHA', | ||
| randomItem: { | ||
| item: 'BACKGROUND', | ||
| color: res.data, | ||
| }, | ||
| }); | ||
| } else { | ||
| alert('뽑기에 실패했습니다(˃̣̣̥ᴖ˂̣̣̥) 관리자에게 문의해주세요'); | ||
| resetModal(); | ||
| } | ||
| } catch (error) { | ||
| setError('HB05'); | ||
| } |
There was a problem hiding this comment.
/pingpong/users/background api 보니까 응답 body가 없어서 res.data가 항상 undefined나 null인 것 같아요.
그래서 지금도 성공한 경우에도 관리자한테 문의하라는 alert이 뜨고 있습니다 ^_ㅜ
instance.patch에서 에러가 나는 경우에는 catch 블록으로 바로 넘어가게 되어서 에러 처리는 catch 쪽에서 해 주면 잘 될 것 같습니다!
Comment on lines
35
to
50
| const res = await instance.patch('/pingpong/users/edge', data); | ||
| if (res.data) { | ||
| setModal({ | ||
| modalName: 'USE-ITEM-GACHA', | ||
| randomItem: { | ||
| item: 'EDGE', | ||
| color: res.data, | ||
| }, | ||
| }); | ||
| } else { | ||
| alert('뽑기에 실패했습니다(˃̣̣̥ᴖ˂̣̣̥) 관리자에게 문의해주세요'); | ||
| resetModal(); | ||
| } | ||
| } catch (error) { | ||
| setError('HB04'); | ||
| } |
There was a problem hiding this comment.
여기도 위에서 리뷰 남긴것처럼 동일하게 catch 블록에서 error.response.data 내용 확인해서 에러처리 하는 식으로 바꿔주셔야 잘 동작할 것 같습니다~!!
PHJoon
approved these changes
Sep 5, 2023
|
뽑기 api응답으로 색상을 받아와서 미리보기를 띄우는데 현재는 응답이 빈 문자열로만 오고 있어서 미리보기가 되지 않습니다! 에러 위치는 catch로 내렸고 에러코드 확인 후 다시 정리할 예정입니다! 그리고 메뉴바와 메인 페이지 userState는 현재 업데이트가 되지 않고 있어서 반영이 안되고 있는데, react-query로 관리하는 방식으로 수정될 예정입니다 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 개요
💻 작업사항
그 외 수정사항
✅ 변경로직