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.

Sign up for GitHub

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

Jump to bottom

[GGFE-223] 코인 업데이트 로직 추가 #972

Merged
merged 1 commit into from
Sep 5, 2023
Merged

[GGFE-223] 코인 업데이트 로직 추가 #972

merged 1 commit into from
Sep 5, 2023

Conversation

yoouyeon
Copy link
Member

📌 개요

  • 구매/선물 이후에 업데이트된 코인을 받아올 수 있도록 했습니다.

💻 작업사항

  • store 페이지 내에서 구매/선물 성공시에 updateCoin(true)를 호출해서 최상위 페이지 컴포넌트에서 새로 coin api를 호출할 수 있게 했습니다.
  • 나중에 업데이트 방식을 react-query로 변경할 가능성도 있습니다 😇

✅ 변경로직

@yoouyeon yoouyeon self-assigned this Aug 31, 2023
hyobb109
hyobb109 approved these changes Sep 5, 2023
View reviewed changes
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.

코인 업데이트 반영 아주 잘 되네요! 고생하셨습니다👍

PHJoon
PHJoon approved these changes Sep 5, 2023
View reviewed changes
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.

구매 선물했을 때 바로바로 잘 작동하네요!! 고생하셨습니다!! 👍

parksangmin1543
parksangmin1543 approved these changes Sep 5, 2023
View reviewed changes
Copy link
Contributor

@parksangmin1543 parksangmin1543 left a comment

Choose a reason for hiding this comment

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

바로바로 코인수정되는 부분 확인했습니다!

@yoouyeon yoouyeon merged commit f3bc447 into main Sep 5, 2023
1 check passed
@hyobb109 hyobb109 deleted the GGFE-223-코인-내역-업데이트 branch October 30, 2023 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyobb109 hyobb109 hyobb109 approved these changes

@parksangmin1543 parksangmin1543 parksangmin1543 approved these changes

@PHJoon PHJoon PHJoon approved these changes

Assignees

@yoouyeon yoouyeon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yoouyeon @PHJoon @parksangmin1543 @hyobb109