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

weekly/9/issue#179 힌트 구매 시 감소 로그 기록 리팩토링 #180

Merged
merged 4 commits into from
Nov 6, 2024

Conversation

gunGeongun
Copy link
Contributor

@gunGeongun gunGeongun commented Nov 6, 2024

🔎 작업 내용

힌트 구매 시 감소 로그 기록하는 로직이 누락되어있어 추가

이미지 첨부

To Reviewers 📢

체크 리스트

  • 테스트를 작성했습니다.
  • 테스트를 통과했습니다.
  • API 변경사항이 존재합니다.
  • API 호출을 직접 실시하였고, 해당 데이터가 정상적으로 표시됩니다.
  • 기존 코드에 영향을 주는 작업 내용이 존재합니다.
  • 향후 추가적인 작업이 필요한 부분이 있습니다.

➕ 관련 이슈

@gunGeongun gunGeongun added the ♻️ refactor 코드가 내부적 작동하는 방식 수정 label Nov 6, 2024
@gunGeongun gunGeongun self-assigned this Nov 6, 2024
Copy link

github-actions bot commented Nov 6, 2024

Unit Test Results

55 tests  ±0   55 ✔️ ±0   21s ⏱️ +3s
24 suites ±0     0 💤 ±0 
24 files   ±0     0 ±0 

Results for commit 70f5626. ± Comparison against base commit ba632b4.

♻️ This comment has been updated with latest results.

@hjinshin hjinshin merged commit f88ac57 into Weekly-9 Nov 6, 2024
3 checks passed
@hjinshin hjinshin deleted the weekly/9/issue#179 branch November 6, 2024 15:19
hjinshin added a commit that referenced this pull request Nov 6, 2024
* Refactor: 힌트 로직 변경

* Refactor: 힌트 로직 변경 테스트 수정

* Refactor: 카카오페이 환경변수명 변경

* Feat: 카카오페이 환경변수 deploy.yml 추가

* Refactor: 그룹 내 랭킹 반환값 변경

* Refactor: Answer시 ranking 상승 객체 변경

* Refactor: 전체 질문 요청 시 groupID가 1인 질문만 리턴

* Refactor: SSE 연결 시간 변경

* Refactor: Group 랭킹 조회 로직 변경

* weekly/9/issue#179 힌트 구매 시 감소 로그 기록 리팩토링 (#180)

* Refactor: 힌트 구매 시 포인트 감소 로그 로직 추가

* Test: 테스트 수정

* Refactor: decreasePoint 로직 수정

* Fix: Group 랭킹 조회 로직 테스트 수정

---------

Co-authored-by: Geon Kim <41728760+gunGeongun@users.noreply.github.com>
hjinshin added a commit that referenced this pull request Nov 6, 2024
* Refactor: 힌트 로직 변경

* Refactor: 힌트 로직 변경 테스트 수정

* Refactor: 카카오페이 환경변수명 변경

* Feat: 카카오페이 환경변수 deploy.yml 추가

* Refactor: 그룹 내 랭킹 반환값 변경

* Refactor: Answer시 ranking 상승 객체 변경

* Refactor: 전체 질문 요청 시 groupID가 1인 질문만 리턴

* Refactor: SSE 연결 시간 변경

* Refactor: Group 랭킹 조회 로직 변경

* weekly/9/issue#179 힌트 구매 시 감소 로그 기록 리팩토링 (#180)

* Refactor: 힌트 구매 시 포인트 감소 로그 로직 추가

* Test: 테스트 수정

* Refactor: decreasePoint 로직 수정

* Fix: Group 랭킹 조회 로직 테스트 수정

* Feat: 카카오페이 api의 partner_user_id에 userId 삽입

* Feat: 카카오페이 api 에러 메세지 출력

---------

Co-authored-by: Geon Kim <41728760+gunGeongun@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 코드가 내부적 작동하는 방식 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants