공통 피드백

• 코드레벨
  • 너무 많은 연관관계를 갖고 있습니다. 도메인 엔티티에서 여러 엔티티를 참조하면서 불편한 점은 없으셨나요? 이 프로젝트에서 대표 도메인들은 어떤 게 있을까요?
    • 문제가 있었다면 어떻게 해결할 수 있을까요
    • 연관을 쪼개고 객체가 아니라 그냥 id로 넣는 방식도 가능함.
  • 현재 LocalDateTime 를 사용하는부분이 있는데, 현재 어떤 기준인가요? (UTC? KST?) 또 시간기준 설정은 어디에 되어있나요?
  • 데이터베이스 스키마를 관리안하시는 걸로 보이는데요. 운영 데이터베이스는 테이블 생성, 수정등이 어떻게 진행될까요?
  • 데이터베이스의 스키마를 못봐서 여쭤봅니다! 인덱스가 적용되어있을까요?
  • 패키지를 도메인레벨로 구축하신 것 같은데요. image, error, auth 이런 것도 도메인이라고 부를 수 있을까요? 분리할 수 있는 방법은 없을까요?
    • 멀티 모듈 / Gradle 단위로 모듈 분리
  • 전반적인 테스트코드 부재 😇
  • 최근에는 외래키를 사용하지 않는 추세
    • 외래키 없어도 조인 가능
    • 외래키 넣기 않고 자주 쓰이는 것에 인덱스 넣기
  • 코드 리뷰
    • 시간 없을 때는 라이브 코드 리뷰
• 문화
  • PR에서 코드가 머지되는 기준이 무엇이었나요?

2조 (공유 다이어리! 동건님팀)

• 인프라
  • 코드디플로이를 사용하신 이유가 궁금합니다
    • Github Action에서 빌드 되고 스크립트 실행하기 위해 사용함.
  • 도커를 사용한 이유가 궁금합니다
  • health check api를 열어 놓으셨는데, 사용하고 계신건지 궁금합니다!
    • 코드 디플로이에 health check hook이 있는 걸로 알고 있음.
  • ci/cd 구축 됐다고 전달 받았는데, 어떤 프로세스로 지속적 통합이 되었고 지속적 제공이 되는지 궁금하네요~
    • 테스트 코드 도입, 테스트 커버리지 등
    • PR에서 빌드 하고 나서 merge 가능한 코드인지 확인

9조 (공방 예약 서비스! 지훈님팀)

• 코드레벨
  • 예약도메인이 너무 빈약합니다. 가령 사용자가 과거의 예약을 조회할 때, onday class를 참조할텐데 onday class가 변경되면 이전에 예약한 정보도 바뀌는건가요?
  • 권한을 별도의 엔티티로 분리하신 이유가 있나요? 또 멤버당 1권한으로 설정하시는데 이 부분에 문제는 없을까요? (권한 데이터를 볼 수가 없어서 여쭤봅니다)
  • 원데이클래스 이미지와 게시글의 이미지가 공통으로 사용되는부분이 있지 않을까요?
• 인프라
  • 웹 서버 (nginx, apache 등)을 고려해보셨나요?
  • 배포는 어떤 프로세스로 이뤄지는지 궁금합니다