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.

Sign up for GitHub

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

Jump to bottom

[refactor] PlaceDetailInfo의 placeLikes에 리뷰 좋아요 수를 반영해요 #172

Closed
1 task done
dong-yxxn opened this issue Nov 14, 2024 · 0 comments
Closed
1 task done

[refactor] PlaceDetailInfo의 placeLikes에 리뷰 좋아요 수를 반영해요 #172

dong-yxxn opened this issue Nov 14, 2024 · 0 comments

Comments

@dong-yxxn
Copy link
Contributor

dong-yxxn commented Nov 14, 2024 •
edited
Loading

Issue

✨ 기능 리스트

  • PlaceDetailInfo의 placeLikes에 리뷰의 좋아요 싫어요 수를 계산해서 반영해요

✏ 사용 이유

가게 상세페이지에서 좋아요, 싫어요 수를 알기 위해서
@dong-yxxn dong-yxxn changed the title [refactor] LikedPlace Domain Test를 만들어요 [refactor] PlaceDetailInfo의 likes에 리뷰 좋아요 수를 반영해요 Nov 14, 2024
@dong-yxxn dong-yxxn changed the title [refactor] PlaceDetailInfo의 likes에 리뷰 좋아요 수를 반영해요 [refactor] PlaceDetailInfo의 placeLikes에 리뷰 좋아요 수를 반영해요 Nov 14, 2024
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
[feat]: Review likes, dislikes 개수 조회 구현
362aea2 
- Place마다 likes와 dislikes의 수를 조회할수있게 했어요

관련 이슈: #172
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
[refactor]: PlaceDetailInfo의 PlaceLikes에 리뷰의 좋아요/싫어요 수를 반영했어요
80c6fd1 
- PlaceDetailInfo의 PlaceLikes에 placeId에 대항하는 리뷰의 좋아요/싫어요 수를 반영하도록 변경했어요.

관련 이슈: #172
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
[feat]: likes dislikes 개수 조회 쿼리 테스트 구현
2baf068 
- ReviewRepository의 countByPlaceIdAndIsLikedTure/False 메서드가 정상 작동하는지 테스트했어요.

관련 이슈: #172
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
[refactor]: PlaceDetailInfo 변경에따라 테스트 코드를 수정했습니다.
afe43b2 
관련 이슈: #172
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
[refactor]: 다른 repository 의존성을 제거했습니다.
52da6fc 
관련 이슈: #172
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
Merge remote-tracking branch 'origin/weekly/11' into refactor/#172-Pl…
db08934 
…aceDetailInfo
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
[refactor]: 필요없는 annotation 제거
c509249 
관련 이슈: #172
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
[refactor]: 데이터를 실제 Db에 반영하기 위해 flush() 사용
4506bea 
- 데이터가 db에 저장되지 않는 오류가 발생하여 flush() 적용하여 해결했습니다.

관련 이슈: #172
dong-yxxn added a commit that referenced this issue Nov 14, 2024
@dong-yxxn
Merge pull request #183 from kakao-tech-campus-2nd-step3/refactor/#172-…
ad92043 
…PlaceDetailInfo

[Refactor] #172 PlaceDetailInfo에 Review 좋아요, 싫어요수를 반영했어요
sanghee0820 added a commit that referenced this issue Nov 14, 2024
@sanghee0820
Merge pull request #190 from kakao-tech-campus-2nd-step3/refactor/#172-…
9d08d39 
…PlaceDetailInfo

[fix]: ReviewRepositoryTest 코드 오류 해결
sanghee0820 added a commit that referenced this issue Nov 14, 2024
@sanghee0820
Merge pull request #201 from kakao-tech-campus-2nd-step3/refactor/#172-…
9aee223 
…PlaceDetailInfo

[fix]: ReviewRepoTest 테스트 오류 해결
sanghee0820 added a commit that referenced this issue Nov 14, 2024
@sanghee0820
Merge pull request #204 from kakao-tech-campus-2nd-step3/refactor/#172-…
94f9003 
…PlaceDetailInfo

[fix]: Category Enum명으로 api 호출이 가능하도록 수정했습니다.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dong-yxxn