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

refactor: 꿀조합 랭킹 API 수정 #38

Merged
merged 10 commits into from
Apr 19, 2024
Merged

refactor: 꿀조합 랭킹 API 수정 #38

merged 10 commits into from
Apr 19, 2024

Conversation

Go-Jaecheol
Copy link
Contributor

@Go-Jaecheol Go-Jaecheol commented Apr 7, 2024

Issue

✨ 구현한 기능

  • 꿀조합 랭킹 API response 수정

    • 반환 데이터 3개 -> 4개
    • author : 객체 -> String
    • favoriteCount -> favorite(boolean)
  • 로그인 한 사용자는 favorite에 자신이 누른 좋아요 여부 / 로그인 안한 사용자는 favorite 전부 false로 반환

  • 관련 서비스 테스트 수정 및 추가

📢 논의하고 싶은 내용

X

🎸 기타

  • 로그인 여부에 상관없이 모두 API 접근이 가능해야 하기 때문에 해당 API 경로에 대한 인터셉터랑 전체 ArgumentResolver 로직 수정했습니다.
    (우선은 로그인 안한 경우 게스트를 나타내는 LoginInfo 반환하도록 수정)

⏰ 일정

  • 추정 시간 : 1h
  • 걸린 시간 : 2h

Copy link

github-actions bot commented Apr 7, 2024

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit d523759.

♻️ This comment has been updated with latest results.

Copy link
Member

@70825 70825 left a comment

Choose a reason for hiding this comment

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

인수 테스트에 로그인된 상태로 테스트하는거 하나 추가하고 자잘한거 수정하면 완벽한 것 같아요~
혹시 제가 놓친게 있을지도 모르니 수정하면 다시 리뷰 요청 보내주세요

@Go-Jaecheol Go-Jaecheol requested a review from 70825 April 19, 2024 06:16
Copy link
Member

@70825 70825 left a comment

Choose a reason for hiding this comment

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

커밋이 순서대로 있어서 편하게 코드 읽었네요 더 수정할 부분은 없어보여요 고생하셨슴다 👍👍

@Go-Jaecheol Go-Jaecheol merged commit a58b9ff into develop Apr 19, 2024
3 checks passed
@Go-Jaecheol Go-Jaecheol deleted the feat/issue-37 branch April 19, 2024 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

꿀조합 랭킹 API 수정
2 participants