Feat: 그룹 내 전체 유저 조회 #22
Conversation
| return new GroupResponse(group.getId(), group.getName()); | ||
| } | ||
|
|
||
| @Transactional |
There was a problem hiding this comment.
readonly 클래스 레벨에서 걸려있으니 Transactional 빼도 됩니다!
There was a problem hiding this comment.
#24 처럼 Group 생성하는 기능은 조회하는 메소드가 아니지만 GroupService에서 작성해두어야 할 것 같아요.
service 레이어를 두는 쪽으로 개발하되 조회 여부로 분리하거나(DomainQueryService-readOnly = true, DomainCommandService), 하나의 DomainService에서 method별 요청을 트랜잭션으로 부여할 수 있을 것 같은데.. 어떠신지요 ? 🌵
|
|
||
| // TODO : 그룹에 유저를 가입시킨다. 유저는 멤버로 등록된다. | ||
| // TODO : 그룹에 유저를 가입시킨다. 유저는 멤버로 등록된다. |
There was a problem hiding this comment.
해당 브랜치에서 개발 안하던 부분이라 너무 거슬렸었는데 제가 써둔 거였네요 🤦🏻 추가 수정하겠습니다
|
|
||
| // TODO : 그룹에 유저를 가입시킨다. 유저는 멤버로 등록된다. | ||
| // TODO : 그룹에 유저를 가입시킨다. 유저는 멤버로 등록된다. |
There was a problem hiding this comment.
해당 브랜치에서 개발 안하던 부분이라 너무 거슬렸었는데 제가 써둔 거였네요 🤦🏻 추가 수정하겠습니다
| return new GroupResponse(group.getId(), group.getName()); | ||
| } | ||
|
|
||
| @Transactional |
There was a problem hiding this comment.
#24 처럼 Group 생성하는 기능은 조회하는 메소드가 아니지만 GroupService에서 작성해두어야 할 것 같아요.
service 레이어를 두는 쪽으로 개발하되 조회 여부로 분리하거나(DomainQueryService-readOnly = true, DomainCommandService), 하나의 DomainService에서 method별 요청을 트랜잭션으로 부여할 수 있을 것 같은데.. 어떠신지요 ? 🌵
🚀 Related Issue
close: #15
📌 Tasks
📚 Remarks