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] 특정 옵션 좋아요 삭제 #87

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

[✨feat] 특정 옵션 좋아요 삭제 #87

wants to merge 22 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?

  • 주요 변경사항을 상세히 기술하세요. (예: 새로운 기능 추가, 버그 수정, 코드 리팩토링 등)
    • 특정 옵션 좋아요 삭제 로직 구현
    • 좋아요 삭제 관련 쿼리 및 메소드 작성

🧪 Testing Details

  • 테스트 코드 및 결과: 작성한 테스트 코드와 주요 테스트 케이스를 설명하고, 통과된 테스트 결과를 요약해 주세요.
    • 다음과 같이 테스트 코드 작성하여 통과 하였습니다.
      • 좋아요가 0 이하일 경우 삭제 시도시 예외 발생
      • 정상적으로 좋아요를 삭제

👀 Checkpoints for Reviewers

  • 리뷰 시 확인할 사항:
    • 이전 PR에서 분기된 브랜치이므로, merge 이전까지 이전 PR의 커밋이 포함되어 있습니다.
    • 비즈니스 로직에서 우려되는 부분이 있는지 검토 부탁드립니다.

🎯 Related Issues

0-tae added 22 commits February 26, 2025 00:17
- 해당 Repository 내부에서 예외처리 진행
- MEMBER_NOT_FOUND 상수 클래스 추가
- 해당 Repository 내부에서 예외처리 진행
- 해당 Repository 내부에서 예외처리 진행
- optionId에 해당된 Like 갯수를 구하는 getLikeCountByOptionId 구현
- appointmentMemberId에 해당된 Like 갯수를 구하는 getLikeCountByAppointmentMemberId 구현
- 좋아요를 저장한 뒤, OptionId에 해당하는 좋아요의 갯수를 반환
- JwtInterceptor에서 반환한다 가정
- 값 인자를 메시지에 전달하기 위해 추가
- 0일 때 삭제하려는 경우 예외처리 추가
- 좋아요 삭제 정상 처리 여부 확인
- 좋아요 갯수가 0개일 때 삭제하려는 경우 예외처리 확인
@0-tae 0-tae self-assigned this Feb 25, 2025
@0-tae 0-tae added ✨ feat Something isn't working 🦖 용태 🦖 Further information is requested labels 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