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

[BE/#599] userService 관심사 분리 #605

Merged
merged 20 commits into from
Jan 12, 2024
Merged

Conversation

koomin1227
Copy link
Member

이슈

체크리스트

  • user repository 생성
  • userService 에서 이미지 처리 로직 분리
  • notificationService를 만들고 userService에서 의존성 분리
  • notificationService 테스트 코드 작성

고민한 내용

  • userService에서는 오직 user 와 관련된 일만 처리 하도록 수정했다.
  • 다른 의존관계, 이미지처리나 registration token 처리 등은 다른 service로 분리 한후 컨트롤러에서 불러서 사용하는 방식으로 변경했다.
  • 이렇게 함으로써 userService 가 user 에 대한 로직만 처리할 수 있게 되어 테스트 코드를 짤 때 쉬워지고 유지 보수가 용이 해질 것 이라고 예상한다.

스크린샷

@koomin1227 koomin1227 added the BE BE 관련 label Jan 11, 2024
@koomin1227 koomin1227 requested a review from namewhat99 January 11, 2024 15:10
@koomin1227 koomin1227 self-assigned this Jan 11, 2024
@koomin1227 koomin1227 added the environment 공통 개발 환경 설정 label Jan 11, 2024
@namewhat99 namewhat99 merged commit b15592f into BE Jan 12, 2024
1 check passed
@namewhat99 namewhat99 deleted the BE-SeparateUserService-#599 branch January 12, 2024 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE BE 관련 environment 공통 개발 환경 설정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants