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

컨벤션 #2

Open

K-Diger opened this issue Nov 29, 2023 · 0 comments

Collaborator

K-Diger commented Nov 29, 2023 •

edited

Loading

Controller, Service 계층을 어떻게 가져갈 것인지?

1안. 단일 Service, Controller에 해당 도메인 API 모두 처리하기
2안. 각 역할에 맞는 Service, Controller로 API 분리하기

다수결 원칙에 따르자 - MashUP 행님들께 11/30 오후 20시 투표 결과 개표 예정

코드 작성 원칙

복붙하지말자 사소한거라도 뭐든...

패키지 구조

api 모듈
- controller
  - BettingDto(req, res)
  - BettingController.kt
- applicationservice
  - BettingApplicationService.kt (DependsOn Betting.kt)
domain 모듈
- betting
  - Betting.kt (DTVO)
  - BettingEntity.kt (Persistence)
  - BettingService.kt
  - BettingRepository.kt

외부 모듈

FCM
S3

커밋 메세지

좋은 커밋 메세지 규약을 참고하기

힙하게 영어로 가보기

PR

작업 내용 (좀 자세하게?)

ToDo

Swagger

MkSpace
- goal, goal_giftcon, gifticon, goal_cheering
Diger
- 나머지

EC2, RDS

MkSpace
- EC2
Diger
- RDS

OAtuh, JWT

아직 미정

얘기해볼만한 것 (리팩터링 후보)

연관관계에 대한 Entity는 어떻게 응답할까?
- userId만 응답한다.
- User전체를 응답한다.
- 꼭 필요한 것만 응답한다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment