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

MusicRepository 추가 #387

Merged
merged 6 commits into from
Jan 29, 2024
Merged

MusicRepository 추가 #387

merged 6 commits into from
Jan 29, 2024

Conversation

khw3754
Copy link
Collaborator

@khw3754 khw3754 commented Jan 28, 2024

Issue

Overview

  • MusicRepository 추가
  • Music 에 있는 함수 모두 repository 로 이동
  • service 에 있는 쿼리문 이동
  • 기존에 Music.{함수이름} 으로 호출하던 부분 모두 수정

Screenshot

  • 음악 Post 테스트
image

To Reviewers

  • 이렇게 Repository 들을 만들다보니 Repository 가 생길 때 마다 해당 repository가 필요한 모듈의 Provider에 Repository 를 추가해줘야 하는데 그게 맞나 싶은 부분이 있어요,, 코드가 별로 안깔끔해 보인달까?

* Music module 과 entity 를 위한 MusicRepository 추가
* entity 에 있는 함수들 모두 repository 로 이동
* service 에 있는 쿼리 모두 repository 로 이동
* 기존 Music. 로 호출하는 부분 수정
* MusicRepository 추가로 인한 테스트코드 변경
@khw3754 khw3754 added ✨ feat 기능 개발 🖥 server server labels Jan 28, 2024
@khw3754 khw3754 added this to the 🎶 music player milestone Jan 28, 2024
@khw3754 khw3754 self-assigned this Jan 28, 2024
@khw3754 khw3754 requested a review from sk000801 as a code owner January 28, 2024 07:44
Copy link
Member

@sk000801 sk000801 left a comment

Choose a reason for hiding this comment

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

마찬가지로 로직이 변한 부분은 없어서 어푸룹 합니다!
(user나 auth 부분은 제꺼 토큰 관련 로직 풀리퀘 어푸룹 되면 얹어서 제가 기능 수행할게요!)

오 말씀하신 부분 어떤 말씀하시는지 이해가 가는데, 저는 오히려 서비스/리포지토리/컨트롤러 역할이 확 분리되어 보인다는 점에서 맘에 드는데, 아니면 모듈을 더 쪼개던지, 뭔가 이런 반복성을 줄이는 커스텀 데코레이터에 대해서 알아봐도 괜찮을 것 같아요! 내일 한번 얘기해봅시당 주말인데 고생 많으셨어용! 👍👍👍

@khw3754 khw3754 merged commit 73da6e6 into develop Jan 29, 2024
1 check passed
@khw3754 khw3754 deleted the server/feature/384 branch January 29, 2024 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 기능 개발 🖥 server server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants