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

♻ remove repository dependency #453

Open
wants to merge 5 commits into
base: be/dev
Choose a base branch
from
Open

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Feb 12, 2025

구조 변경에 대해 용이하게 논의하기 위해 repository와 service의 의존성이 무분별하게 섞여 있던 것을 최대한 service로 통일하도록 했습니다.

Recipe는 User의 소유물입니다. 그래서 UserService는 RecipeService로만 동작합니다.
그런데 부득이하게 Recipe가 User를 알아야 할 때(생성할 때 등)가 있습니다. 그런 경우에는 UserRepository를 이용하도록 했습니다.

그래서 소유 관계를 생각하며 확인해주시면 되겠습니다~
관련해서 의견 자유롭게 주세요!

closes #452

Copy link
Contributor Author

github-actions bot commented Mar 6, 2025

Overall Project 90.33% 🍏

There is no coverage information present for the Files changed

Copy link
Contributor Author

Overall Project 90.47% 🍏
Files changed 100% 🍏

File Coverage
IngredientRecipeService.java 100% 🍏
RecipeStepService.java 100% 🍏
IngredientService.java 94.41% 🍏
UserService.java 94.38% 🍏
UserFollowService.java 92.5% 🍏
RecipeService.java 87.84% 🍏

@hyxrxn hyxrxn requested review from tackyu and HaiSeong March 10, 2025 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE Backend ♻️ refactor refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants