w10 코드리뷰 요약

[Kdonghs]

2. [x] 패키지명 수정(DTO->dto, BE->be)
3. [x] parsingPram 오타
4. [x] 프로퍼티스에 code,jwt 관련 정보 yml로 분리
5. [x] Status enum의 명칭 변경(ex) Status -> TaskStatus)
6. [x] 메일 본문 텍스트 구성하는 방식 변경(https://github.com/kakao-tech-campus-2nd-step3/Team1_BE/pull/109#discussion_r1835445419)
7. [x] 멤버 생성시 이메일 중복 예외처리 변경(UNAUTHORIZED 401 -> CONFLICT 409)
8. [x] 프로젝트 종료 여부를 확인하는 도메인 로직을 내부로 캡슐화(https://github.com/kakao-tech-campus-2nd-step3/Team1_BE/pull/109#discussion_r1835445996)
9. [x] Role 검증을 위한 코드를 메소드로 만들어서 캡슐화(https://github.com/kakao-tech-campus-2nd-step3/Team1_BE/pull/109#discussion_r1835447054)
10. [ ] task에 isOwner라는 함수를 만들어서 사용
11. [ ] dto 파일내 함수 분리
12. [ ] 테스트용으로 사용하는 컨트롤러 분리(예: AdminController, DevelopTestController 등)

[Kdonghs]

해결

• 패키지명 수정(DTO->dto, BE->be)
• parsingPram 오타
• 프로퍼티스에 code,jwt 관련 정보 yml로 분리
• Status enum의 명칭 변경(ex) Status -> TaskStatus)
• 메일 본문 텍스트 구성하는 방식 변경(https://github.com/kakao-tech-campus-2nd-step3/Team1_BE/pull/109#discussion_r1835445419)
• 멤버 생성시 이메일 중복 예외처리 변경(UNAUTHORIZED 401 -> CONFLICT 409)

[Kdonghs]

굳이?

• 테스트용으로 사용하는 컨트롤러 분리(예: AdminController, DevelopTestController 등)

[sunandrabbit]

https://github.com/kakao-tech-campus-2nd-step3/Team1_BE/pull/58#discussion_r1797699763 6주차 내용인데 이부분은 어떻게 할까요?

[Kdonghs]

그것도 그건데 프로젝트 서비스에 권한 검증하는 코드가 하나도 없어(멤버의 토큰으로 프로젝트 생성가능해짐)

[Kdonghs]

#58 (comment) 6주차 내용인데 이부분은 어떻게 할까요?

중복되는 구조를 해결하기에는 사이즈가 너무 크고, 서비스 레이어에서 req를 참조하는 사항은 컨트롤러로 옮겨서 파싱하는 방식으로 하면 될거 같은데? ProjectOptionController 처럼

[Kdonghs]

일단 수정사항 위클리에 반영함(11.11 02:15시)