Week3 수정사항

[Kdonghs]

공통

• 디렉토리 이름 변경
• oauth2-client, security를 꺼 놓으면 인증요구를 하지 않음(필요시 꺼놓고 사용하다 커밋에는 켜 주세요)

권순호

• MemberController에서 deleteMember의 반환값 수정(SingleResult로 반환)
• AttendUrlController에서 generateUrl의 반환값 수정(SingleResult로 반환)
• AttendUrlResponseDTO는 생성자로 값 반영(setter 사용 지양)
• RequestMapping 수정(모든 경로는 소문자로 작성 및 /api로 시작하게금 변경)(AttendUrlController, MemberController)
• MemberService의 생성자에 @Autowired적용

김도헌

• ProjectController에서 deleteProject의 반환값 추가(SingleResult로 삭제한 id같은걸 반환)
• @GetMapping("/{project-id}")로 변경하는게 가독성에서 유리함(피드백 사항)
• ProjectService의 생성자에 @Autowired적용
• createProject mapper로 뺴서 적용
• createProject ProjectDTO생성자에 getGuests와 getOptions을 빈 리스트로 초기화하면 if로 검사하지 않아도 됨
• projectDTO에서 viewType의 Ojbect대신 다른 자료형 적용(피드백 사항)
• ProjectService에서 createProject @transactional 적용(피드백 사항)
• view_type이 object여서 hibernate가 sql을 생성하지 못함
• project에 task 1:1 매핑 없음
• addGuest, addOption으로 매핑을 추가하는 방식은 없음(새로운 게스트,옵션을 생성할떄 프로젝트에 매핑해서 사용함)

김동혁

• 예외처리 텍스트 수정

조서영

• TaskController에서 createTask의 반환값 추가(SingleResult로 삭제한 id같은걸 반환)(피드백 사항)
• createTask의 이름 변경(createTask로 하던가 deleteTake로 하던가)
• @Valid추가
• Long project_id, Long task_id 보다 카멜 케이스를 사용하는 것이 좋습니다.(피드백 사항)
• TaskService의 생성자에 @Autowired적용
• deleteTask삭제시 존재 여부 및 예외처리 코드 필요(객체 조회하면서 존재 검증 후 taskRepository.delete(entity); 로 삭제)
• task엔티티에 태스크 시작일, 종료일 없음
• task엔티티에 @column 없음
• TaskRepository에서 deleteById를 정의 하는 파라미터는 1개인데 2개 요청중(Long projectId, Long taskId)

[sunandrabbit]

오 피드백 좋아요.
추가로 제가 conflict resolve 할때 제가 임의로 만들었던 Entity를 제거하고 원래 작성하기로 하시던 분들 Entity로 바꿔놨습니다.
그래서 나중에 연관관계 매핑등은 새로 협의해서 다시해야 될거 같아요. 그리고 정윤님이 작성하신 API 문서토대로 Entity 필드들 수정할 예정입니다.

[sunandrabbit]

추가로 BaseEntity 삭제 할까요?

[Kdonghs]

BaseEntity는 굳이 삭제 안해도 될 듯

[Kdonghs]

너무 모든걸 맞추고 시작하려 하지 않아도 됨
어차피 스프린트3가서 정리해도 될 문제임

[sunandrabbit]

확인 했음

[suno-boy]

확인 했숩니당

[seoyoung-dev]

확인했습니다!

[sunandrabbit]

project에 task 1:1 매핑 없음
이부분에서 project가 여러개의 task를 가지지 않나요?
따라서 project : task = 1:N 관계여야 되는 것 같다고 생각합니다

[sunandrabbit]

그리고 Project에서 viewType이 API 설계봐도 없고, 뭐하는 필드인지 몰라서 일단 제거 했습니다.

[sunandrabbit]

addGuest, addOption으로 매핑을 추가하는 방식은 없음(새로운 게스트,옵션을 생성할떄 프로젝트에 매핑해서 사용함)
이부분에서 그러면 Project가 가지고있는
List guests, List options를 통해 양방향 매핑이 필요 없나요?