✍🏻 주요 안건

김민기 피드백

• Feedback

이동엽 피드백

• Feedback

📑 회의 내용

코드리뷰 내용

• Global
  • 예외 처리 로직에서 GlobalExceptionHandler 에 private 메서드 만들고, CommonResponse 반환하게 바꾸기
  • CommonResponse 에서 속성명 dataResponse → data 로 수정하기
  • ExceptionExectutor 에서 메서드명 파스칼 케이스로 바꾸기
  • Business Exception에서 그냥 ErrorType을 생성자 매개변수로 받도록 수정하자.
    • • 생성자도 protected로 수정하기
  • BaseTimeEntity
    • create, update 메서드 지워보기 → 어노테이션으로 해놨으니 작동할거같다. ✔️
  • AccessToken 생성시 Member를 넘기지 말고, 필요한 값들만 넘기자.
• Auth
  • 테스트 코드 짜기
  • AuthService에서 비밀번호 비교 로직 custom validation으로 바꾸기 : 링크 9.2 → @Donggwon Seo
• Board
  • Web Layer 단위 테스트도 추가 작성하기 → @WebMvcTest = 각자 ✔️
  • ControllerTest 클래스 명 수정하기 → IntegrationTest = O
  • 게시글 작성 테스트 → title 말고 id를 생성하자. = 윤진원 ✔️
  • 서비스도 추상화하기 = 각자 ✔️
  • JPA Batch Insert 적용하기 = 이동엽 ✔️
  • 게시글 작성시 = 윤진원 ✔️
    • DTO에서 memberId를 받지말고, token에서 추출하자.
    • Spring Arguments Resolver를 적용하기 → JwtProvider에서 extract도 구현해야겠다.
  • 게시글 조회시 조회수 증가하니까, @Transactional 붙이기 = 이동엽 ✔️
  • 이미지 저장에서 forEach 문 지우고, toList 로 마무리하기. = 윤진원 ✔️
  • url “/api” prefix 빼기 = O

주요 안건2

• Back-end Class 네이밍 컨벤션 → 다들 참고하고 코드짜기 [동사를 앞으로]

주요 안건3

새로 정한 규칙들

• DTO ↔ Entity 간에 Mapper를 만들기. = 각자 ✔️
  • → 테스트 코드에도 모두 적용하고, DTO 사용 없애기.
  • → Board는 완료 ✔️
  • → Member는 아직
• assertEquals() vs assertThat() 둘 중 하나로 통일을 하자. = 윤진원
  • → @Jinwon Yoon 알아오기