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

[✨feat] 특정 옵션 좋아요 추가 #86

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from
Open

[✨feat] 특정 옵션 좋아요 추가 #86

wants to merge 16 commits into from

Conversation

0-tae
Copy link
Contributor

@0-tae 0-tae commented Feb 25, 2025

🚀 What’s this PR about?

  • 작업 내용 요약:
  • 특정 옵션 좋아요 추가 API 구현 하였습니다.

🛠️ What’s been done?

  • 주요 변경사항을 상세히 기술하세요. (예: 새로운 기능 추가, 버그 수정, 코드 리팩토링 등)
    • 특정 옵션 좋아요 추가 서비스 로직 API 구현
    • LikeRepsoitory에 optionId에 대한 좋아요 카운트 수를 반환하는 메소드 추가
    • LikesCount VO 삭제 및 컬럼 삭제
    • 좋아요 생성에 필요한 레포지토리에 getById default 메소드 추가

🧪 Testing Details

  • 테스트 코드 및 결과: 작성한 테스트 코드와 주요 테스트 케이스를 설명하고, 통과된 테스트 결과를 요약해 주세요.
    • 좋아요 추가에 관련된 LikeServiceImpl 로직 테스트 코드 작성 및 테스트 코드 통과
    • 최대 좋아요 개수를 초과하는 테스트 통과

👀 Checkpoints for Reviewers

  • 리뷰 시 확인할 사항:
    • API에서 Path Param이 적절한지 확인 부탁드립니다.
    • LikeCounts VO 제거 하였습니다. Like 클래스 명을 리팩토링한 관계로 QLikes -> QLike로 바꾸게 되었으니 인지 해주시면 좋을 것 같습니다.

🎯 Related Issues

0-tae added 16 commits February 26, 2025 00:17
- 해당 Repository 내부에서 예외처리 진행
- MEMBER_NOT_FOUND 상수 클래스 추가
- 해당 Repository 내부에서 예외처리 진행
- 해당 Repository 내부에서 예외처리 진행
- optionId에 해당된 Like 갯수를 구하는 getLikeCountByOptionId 구현
- appointmentMemberId에 해당된 Like 갯수를 구하는 getLikeCountByAppointmentMemberId 구현
- 좋아요를 저장한 뒤, OptionId에 해당하는 좋아요의 갯수를 반환
- JwtInterceptor에서 반환한다 가정
- 값 인자를 메시지에 전달하기 위해 추가
@0-tae 0-tae added ✨ feat Something isn't working 🦖 용태 🦖 Further information is requested labels Feb 25, 2025
@0-tae 0-tae self-assigned this Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat Something isn't working 🦖 용태 🦖 Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant