[Feat/#81] 전체 수신거부 API 1차 구현 #88
Conversation
| @RequestMapping("/api/v1/workbooks") | ||
| @RequestMapping("/api/v1/") |
There was a problem hiding this comment.
WorkbookSubscriptionController인데 /workbooks를 제거한 이유가 있나요?
추가로 /를 남겨둔 이유가 잇나요?
There was a problem hiding this comment.
해당 컨트롤러의 API가 다 workbooks로 시작하진 않아서 일단 뺐습니다
| import org.springframework.http.HttpStatus | ||
| import org.springframework.validation.annotation.Validated | ||
| import org.springframework.web.bind.annotation.* | ||
|
|
||
| @Validated | ||
| @RestController | ||
| @RequestMapping("/api/v1/workbooks") | ||
| @RequestMapping("/api/v1/") | ||
| class WorkbookSubscriptionController( |
There was a problem hiding this comment.
WorkBookSubscriptionController이 아니라 그냥 SubscriptionController로 하는건 어떤가요?
| @Service | ||
| class MemberService { | ||
|
|
||
| fun getMemberId(dto: GetMemberIdDto): Long { |
There was a problem hiding this comment.
저는 단수 조회는 read, 복수 조회는 browse를 사용하는데 어떠신가요?
get도 충분히 이해할 수 있어서 통일할 필요는 없지만 코멘트 남깁니다
| fun updateDeletedAtInAllSubscription(command: UpdateDeletedAtInAllSubscriptionCommand) { | ||
| dslContext.update(SUBSCRIPTION) | ||
| .set(SUBSCRIPTION.DELETED_AT, LocalDateTime.now()) | ||
| .set(SUBSCRIPTION.UNSUBS_OPINION, command.opinion) // TODO: opinion row 마다 중복 해결 |
There was a problem hiding this comment.
opinion row 마다 중복 해결
이게 어떤 문제인가요?
There was a problem hiding this comment.
전체 구독 내역에 업데이트(여러개 row에 업데이트)하기 때문에 요청으로 들어온 입력(구독취소 사유)이 해당 로우들에 동일하게 업뎃됨
There was a problem hiding this comment.
이건 뭐.. 어쩔 수 없지 않을까요..?
대신 dslContext.batchUpdate()로 배치 업데이트만..
| class MemberService { | ||
|
|
||
| fun readMemberId(dto: readMemberIdDto): Long { | ||
| return 1L // TODO: implemenets |
| ) { | ||
|
|
||
| @PostMapping("{workbookId}/subs") | ||
| @PostMapping("workbooks/{workbookId}/subs") |
There was a problem hiding this comment.
20번째 라인의 매핑은 /로 끝나는데 해당 매핑은 /로 안끝나니까 햇갈릴 수 있을꺼 같은데 통일하는게 어떤가요?
🎫 연관 이슈
resolved #81
💁♂️ PR 내용
🙏 작업
🙈 PR 참고 사항
📸 스크린샷
🤖 테스트 체크리스트