- 
                Notifications
    
You must be signed in to change notification settings  - Fork 0
 
[Refactor]면접 예약 날짜 변경 기능 추가 #93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The head ref may contain hidden characters: "DASOMBE-18-refactor-\uBA74\uC811-\uC608\uC57D-\uB0A0\uC9DC-\uBCC0\uACBD-\uAE30\uB2A5-\uCD94\uAC00"
Conversation
4014fb4    to
    4c5cc6f      
    Compare
  
    There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
작업 고생하셨습니다! 추가 피드백 드리겠습니다~
- 이메일 인증코드 저장 방식
현재 메모리에 저장하는 방식은 서버 재시작 시 데이터가 유실되고, 다중 서버 환경에서 일관성을 보장할 수 없습니다. 따라서 인증코드는 외부 DB에 저장하도록 하는 것이 적절합니다. 특히 Redis를 사용하면 TTL 설정을 통해 자동 만료 처리가 가능하고, 다중 인스턴스 서버 환경(분산 시스템)에서도 외부 DB에 데이터를 의존하기 때문에 인증 로직의 일관성을 유지할 수 있습니다.
JWT를 저장하는 방식이 Redis로 구현되어 있으므로 참조해보세요. - 커밋 메시지 및 PR 컨벤션 준수
커밋 메시지와 PR 내용은 팀 컨벤션에 맞추어 작성해주세요. 일관된 컨벤션은 변경 이력을 추적하고 코드 리뷰 효율을 높일 수 있습니다. - EmailService가 domain/email, domain/google 두 곳에 중복되고 있는데, domain/google에 위치한 EmailService로 로직을 통일시켜주세요.
 
        
          
                src/main/java/dmu/dasom/api/domain/common/exception/ErrorCode.java
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                src/main/java/dmu/dasom/api/domain/interview/repository/InterviewSlotRepository.java
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                src/main/java/dmu/dasom/api/domain/interview/service/InterviewServiceImpl.java
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                src/main/java/dmu/dasom/api/domain/recruit/service/RecruitService.java
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                src/main/java/dmu/dasom/api/domain/recruit/service/RecruitServiceImpl.java
              
                Outdated
          
            Show resolved
            Hide resolved
        
      | 
           @ysw789 컨벤션에 맞춰 제대로 작성하지 못한점 죄송합니다..  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- RecruitController에 발생할 수 있는 에러 코드 명세 추가해주세요
 - 병합 충돌 해결 후 push해주세요
 
| 
           @ysw789 충돌 해결했습니다  | 
    
[refactor] 면접 예약 날짜 변경 기능 추가
Issue
변경 내용
체크리스트
npm run lintornpm run lint:fix)npm run build)테스트
리뷰 요구사항