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

[GGFE-205] 유저 보유 코인 조회 api 연결 #956

Merged
merged 2 commits into from
Aug 24, 2023

Conversation

yoouyeon
Copy link
Member

📌 개요

  • 유저 보유 코인 조회 api 연결했습니다.

💻 작업사항

  • 기존 mock api로 되어있던 코인 api를 실제 api로 변경했습니다.
  • 상점 페이지 내에서 코인을 사용하는 경우 코인이 줄어든 내역을 새로 받아올 수 있게 state를 만들었습니다. 코인을 사용한 뒤에 setUpdateCoinupdateCoin 값을 true로 설정하면 코인을 새로 받아옵니다.
    (서버 측에 업데이트 된 매치 정보를 다시 받아오는 로직도 reloadMatchState라는 리코일로 관리하고 있어서 코인 업데이트 방식도 동일하게 프론트에서 관리하는 식으로 처리했는데 나중에 여유 되면 react-query 로 관리하는 쪽으로 바꾸는 것도 좋을 것 같습니다 ^_ㅜ)

✅ 변경로직

@yoouyeon yoouyeon self-assigned this Aug 24, 2023
Copy link
Contributor

@PHJoon PHJoon left a comment

Choose a reason for hiding this comment

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

보유 코인 정보 잘 받아오네요! 코인 정보 업데이트하는 부분은 적어두신 것처럼 react-query 적용하면 더 좋을 것 같네요! 고생하셨습니다!! 👍👍

@@ -44,7 +44,7 @@ export function StoreModeWrap({
매뉴얼
</button>
<div className={styles.coins} onClick={viewCoinHistory}>
{coin.coin}
{coin.coin.toLocaleString()}
Copy link
Member

Choose a reason for hiding this comment

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

숫자 , 찍는 법 배워갑니다👍

Copy link
Member

@hyobb109 hyobb109 left a comment

Choose a reason for hiding this comment

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

보유 코인 잘 불러오는 것 확인했습니다! 부자가 된 기분이네요. 저도 코인 업데이트는 react-query로 관리하는 것이 좋아보입니다👍 고생하셨습니다!!

@yoouyeon yoouyeon merged commit ad872bf into main Aug 24, 2023
@PHJoon PHJoon deleted the GGFE-205-유저-보유-코인-조회-api-연결 branch September 4, 2023 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants