w10 코드리뷰 요약

[Kdonghs]

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

[Kdonghs]

해결

• 패키지명 수정(DTO->dto, BE->be)
• parsingPram 오타
• 프로퍼티스에 code,jwt 관련 정보 yml로 분리
• Status enum의 명칭 변경(ex) Status -> TaskStatus)
• 메일 본문 텍스트 구성하는 방식 변경(10주차 리뷰 부탁드립니다. #109 (comment))
• 멤버 생성시 이메일 중복 예외처리 변경(UNAUTHORIZED 401 -> CONFLICT 409)

[Kdonghs]

굳이?

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

[sunandrabbit]

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

[Kdonghs]

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

[Kdonghs]

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

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

[Kdonghs]

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