동아리 생성, 동아리 상세 정보 수정 API 구현 #124
Conversation
dh2906
left a comment
dh2906
left a comment
There was a problem hiding this comment.
고생하셨습니다! 앞으론 PR 작성하시고 깃허브에 리뷰 요청 버튼 눌러주시면 됩니당
깃허브 코파일럿도 사용이 되는지는 모르겠는데 얘한테 먼저 리뷰 받으셔도 돼요.
동아리 생성은 일단 보류하고 동아리 정보 수정 관련해서만 작성해주시면 될 것 같아요.
몇 가지 코멘트 확인 후 수정해주신 리뷰 요청 부탁드릴게요~!
.gitignore
Outdated
| !gradle/wrapper/gradle-wrapper.jar | ||
| !**/src/main/**/build/ | ||
| !**/src/test/**/build/ | ||
| src/main/resources/config/ |
There was a problem hiding this comment.
yml 파일의 경우 private 레포에서 서브 모듈로 관리하고 있어 gitignore를 지정하면 안됩니다!
.git/info/exclude에 해당 설정을 넣어 로컬에서만 포함하지 않도록 바꿔주세요
There was a problem hiding this comment.
아 제 로컬에서만 다른건 또 이렇게 설정해야 하는군요! 수정하겠습니다
| - NOT_FOUND_USER (404): 유저를 찾을 수 없습니다. | ||
| """) | ||
| @PostMapping | ||
| ResponseEntity<ClubDetailResponse> createDetail( |
There was a problem hiding this comment.
동아리 생성의 경우 관리자에게 권한을 승인받은 뒤 만들 수 있도록 계획 중입니다.
또한, 서비스 초기엔 한기대를 대상으로 하기에 초기 데이터는 저희 쪽에서 직접 다 넣어 줄 예정입니다.
There was a problem hiding this comment.
아 그러면 이런 동아리 생성과 관련된 부분은 구현을 보류하는 것이 좋을까요??
There was a problem hiding this comment.
넵 일단 보류하는게 좋을 것 같습니다. 이에 대해 상의해야 할 내용도 있을거라
| - NOT_FOUND_USER (404): 유저를 찾을 수 없습니다. | ||
| """) | ||
| @PutMapping("/{clubId}") | ||
| ResponseEntity<ClubDetailResponse> updateDetail( |
There was a problem hiding this comment.
| ResponseEntity<ClubDetailResponse> updateDetail( | |
| ResponseEntity<ClubDetailResponse> updateClubDetail( |
이런 네이밍은 어떠신가요?
There was a problem hiding this comment.
말씀해주신 것 두 개중에서 고민하긴 했었는데
다시 보니 언급해주신걸로 수정해야 좀더 어떤 메소드인지 구체적으로 알 수 있을 것 같습니다
수정하겠습니다!
| @Schema(description = "동아리 상세 소개", example = "BCSD에서 얻을 수 있는 경험\n1. IT 실무 경험", requiredMode = Schema.RequiredMode.REQUIRED) | ||
| @NotBlank(message = "상세 소개는 필수 입력입니다.") | ||
| String introduce, |
There was a problem hiding this comment.
동아리 한 줄 소개와 상세 소개는 서로 다른 페이지에서 수정이 이뤄집니다.
피그잼 1월 6일 페이지 관리 쪽 확인 부탁드릴게요!
There was a problem hiding this comment.
아 그렇네요!
그러면 다른 페이지에서 수정이 이루어지는 것들은 서로 다른 메소드(controller 부분)를 호출하도록 하는 것이 좋을까요??
| @Override | ||
| public ResponseEntity<ClubDetailResponse> createDetail( | ||
| @Valid @RequestBody ClubDetailCreateRequest request, | ||
| @UserId Integer userId | ||
| ) { | ||
| ClubDetailResponse response = clubService.createDetail(userId, request); | ||
| return ResponseEntity.ok(response); | ||
| } |
There was a problem hiding this comment.
맨 처음 동아리를 만들 때 상세 정보는 동아리 생성과 같이 있어야 할 것 같아요.
club에서 상세 정보 관련 컬럼들이 모두 Not Null 제약 조건이 들어있어 따로 API를 만들면 동아리 생성이 안될 것으로 보입니당
There was a problem hiding this comment.
아 그러면 이 메소드를 후에 다시 구현하게 된다면,
기존에 상세 정보만 생성하는 것 -> 동아리(club)와 상세 정보 같이 생성하는 메소드로 변경해서 구현하는 것이 좋을까요?
(기존) createDetail -> (변경) createClubAndDetail
| 동아리 가입 신청서를 제출합니다. | ||
| 설문 질문이 없는 경우 answers는 빈 배열을 전달합니다. | ||
There was a problem hiding this comment.
Trailing whitespace should be removed from documentation strings. This is also present on lines 162, 180, and 195.
2 participants
동아리 생성 기능과 상세 정보 수정 기능 구현했습니다.
구현 중에 DTO 두 개(ClubCreateRequest, ClubUpdateRequest) 추가되었습니다.
회의 때도 언급드렸던 것처럼 동아리 소개 이미지 추가 관련은 보류했습니다.
확인 부탁드립니다..!