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

[BSVR-231] 스크랩 개수 저장 로직 추가 #156

Merged
merged 4 commits into from
Aug 22, 2024
Merged

Conversation

pminsung12
Copy link
Collaborator

📌 개요 (필수)

  • 스크랩 개수 저장 로직 추가

🔨 작업 사항 (필수)

  • 리뷰 엔티티에 scrapsCount 컬럼 추가
  • 리뷰 도메인, 응답 객체에 scrapsCount 필드 추가, 매개변수 수정
  • scrap controller, service 코드 수정
  • scrap repository 코드 업데이트
  • review repository에서 scrapsCount 필드 업데이트하는 메서드 추가

⚡️ 관심 리뷰 (선택)

  • 특별히 확인 받고 싶은 내용을 적어주세요.

🌱 연관 내용 (선택)

  • @EunjiShin 갓 은 지의 공감 파트 로직이 똑같아 코드를 참고함. (베낌)
  • dev와 prod 테이블에 아래의 sql문 적용해서 review 테이블에 scraps_count 컬럼 추가
alter table reviews add column scraps_count int DEFAULT 0;

💻 실행 화면 (필수)

image

@pminsung12 pminsung12 self-assigned this Aug 22, 2024
@pminsung12 pminsung12 added ✨ Feature 기능 개발 🚀 Self Approve 빨리 merge가 필요한 경우, 코드 리뷰를 생략했다는 의미로 추가해요. 남용하지 말 것. and removed size/M labels Aug 22, 2024
Copy link

Test Results

51 tests  +1   51 ✅ +1   1s ⏱️ ±0s
17 suites ±0    0 💤 ±0 
17 files   ±0    0 ❌ ±0 

Results for commit 2f718fa. ± Comparison against base commit ac39d25.

This pull request removes 2 and adds 3 tests. Note that renamed tests count towards both.
org.depromeet.spot.usecase.service.review.ReviewScrapServiceTest ‑ 스크랩_개수_확인()
org.depromeet.spot.usecase.service.review.ReviewScrapServiceTest ‑ 스크랩_제거()
org.depromeet.spot.usecase.service.review.ReviewScrapServiceTest ‑ 스크랩_취소()
org.depromeet.spot.usecase.service.review.ReviewScrapServiceTest ‑ 스크랩_토글_제거()
org.depromeet.spot.usecase.service.review.ReviewScrapServiceTest ‑ 스크랩_토글_추가()

@pminsung12 pminsung12 merged commit 5b3bc93 into main Aug 22, 2024
5 checks passed
@pminsung12 pminsung12 deleted the feat/BSVR-231 branch August 22, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🚀 Self Approve 빨리 merge가 필요한 경우, 코드 리뷰를 생략했다는 의미로 추가해요. 남용하지 말 것.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant