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

좋아요 토글에서 좋아요, 좋아요 취소로 엔드포인트 분리 #163

Merged
merged 20 commits into from
Mar 14, 2024

Conversation

yenzip
Copy link
Contributor

@yenzip yenzip commented Mar 13, 2024

🎫 관련 이슈

Resolves #162
Fix #153

✅ 구현 내용

  • 좋아요 토글에서 좋아요, 좋아요 취소 api 엔드 포인트 분리
  • 좋아요 service 및 repository 코드 수정
  • 좋아요 validator 및 errorMessage 추가
  • 좋아요 테스트 수정
  • 좋아요 생성 및 삭제 시 likeCount 값 변경(업데이트)

💬 코멘트

@yenzip yenzip added 🛠 리팩토링했슈 리팩토링 🚨 돌 굴러가유 긴급 리뷰 🥕 더 수정해봤슈 리뷰 후 피드백 반영 ✅ 테스트혀봤슈 빌드&테스트가 필요할 때 태그 labels Mar 13, 2024
@yenzip yenzip added this to the 3차 스프린트 milestone Mar 13, 2024
@yenzip yenzip self-assigned this Mar 13, 2024
@yenzip yenzip linked an issue Mar 13, 2024 that may be closed by this pull request
3 tasks
Copy link

📝 Jacoco Test Coverage

Overall Project 71.4% -0.46%
Files changed 78.08%

File Coverage
SaveProjectRequest.java 100% 🍏
UserProfileResponse.java 100% 🍏
UserSummary.java 100% 🍏
LikeValidator.java 100% 🍏
LikeResponse.java 100% 🍏
LikeService.java 100% 🍏
User.java 92.9% 🍏
LikeController.java 0%

Copy link

📝 Jacoco Test Coverage

Overall Project 71.57% -0.45%
Files changed 83.76% 🍏

File Coverage
SaveProjectRequest.java 100% 🍏
UserProfileResponse.java 100% 🍏
UserSummary.java 100% 🍏
LikeValidator.java 100% 🍏
LikeResponse.java 100% 🍏
LikeService.java 100% 🍏
Project.java 97.95% 🍏
User.java 92.9% 🍏
LikeController.java 0%

@yenzip
Copy link
Contributor Author

yenzip commented Mar 13, 2024

커밋 롤백하다가 세희 언니 커밋 기록이 섞인 것 같네요... 😢

Copy link
Member

@Sehee-Lee-01 Sehee-Lee-01 left a comment

Choose a reason for hiding this comment

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

정말 백엔드에서 이야기가 많은 기능인데 확장성 고려해주시면서 기능 구현하신 것이 보입니다! 꼼꼼히 구현해주셔서 감사합니당!! 🥰

참고로 이제 LikeId로 삭제를 요청하니 상세조회 때 해당 프로젝트에 대한 LikeResponse도 리스트로 같이 보내줘야 겠네용! 확인차 맞는지 여쭤봅니당!!

import sixgaezzang.sidepeek.users.domain.User;
import sixgaezzang.sidepeek.users.repository.UserRepository;
import sixgaezzang.sidepeek.users.service.UserService;

@Service
@RequiredArgsConstructor
Copy link
Member

Choose a reason for hiding this comment

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

나중에 프로젝트 상세조회에서 Like 같이 보내줄 때 @Transactional(readOnly=true) 설정이 추가될 수도 있겟네유!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

맞습니다! 현재는 생성과 삭제만 존재하기 때문에 우선 제외시켰긴 합니다!

@yenzip
Copy link
Contributor Author

yenzip commented Mar 14, 2024

정말 백엔드에서 이야기가 많은 기능인데 확장성 고려해주시면서 기능 구현하신 것이 보입니다! 꼼꼼히 구현해주셔서 감사합니당!! 🥰

참고로 이제 LikeId로 삭제를 요청하니 상세조회 때 해당 프로젝트에 대한 LikeResponse도 리스트로 같이 보내줘야 겠네용! 확인차 맞는지 여쭤봅니당!!

이 부분은 훈오님과 이야기 나눠봐야 할 것 같아요! 아직까진 말씀이 없으셨습니다! merge 되는 대로 한 번 이야기 나눠볼게요! 감사합니다 ❤️

@yenzip yenzip merged commit ee80972 into dev Mar 14, 2024
2 checks passed
@Sehee-Lee-01 Sehee-Lee-01 deleted the feat/#162-seperate-likes branch March 14, 2024 04:12
@Sehee-Lee-01 Sehee-Lee-01 restored the feat/#162-seperate-likes branch March 14, 2024 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ 테스트혀봤슈 빌드&테스트가 필요할 때 태그 🐛 아이고 벌레! 버그 🚨 돌 굴러가유 긴급 리뷰 🛠 리팩토링했슈 리팩토링 🥕 더 수정해봤슈 리뷰 후 피드백 반영
Projects
None yet
2 participants