Skip to content
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

fix: 주기정보저장시 차량정보의 위경도값 저장하도록 수정 #285

Conversation

Suxxxxhyun
Copy link
Collaborator

@Suxxxxhyun Suxxxxhyun commented Feb 8, 2025

🔧 어떤 작업인가요?

주기정보저장시 차량정보의 위경도값 저장하도록 수정

#️⃣ 연관된 이슈

#271

💡 리뷰어에게 하고 싶은 말

[기존 데이터 타입 : 기본타입 -> 참조타입으로 변경]

  • 데이터가 null인 경우도 고려하여 참조타입으로 변경하였습니다.

[주기정보저장시 차량정보의 위경도값저장하도록수정]

  • Consumer에서 주기정보 관련하여 데이터를 받을 때, 주기정보의 데이터 개수가 60개가 아닌지 확인
    -> 60개가 아니라면, 예외처리하도록 코드 수정하였습니다.
  • 주기정보 저장시, 최신의 주기정보를 토대로 차량테이블의 위,경도 값을 저장하도록 하였습니다.
  • 주기정보 저장시 제대로 저장이 안되었을 경우, 차량의 위,경도 값도 롤백이 됨을 확인하였습니다.

[docker-compose에 redis관련 설정 추가 -> 해당 부분 주석 처리]

  • 관제 서버 실행시킬때 redis가 없어서 에러생기는 이슈로 인해 docker-compose.yml에 redis관련 설정을 추가하였습니다.
  • 로컬에 redis를 설치하고 사용하는 분도 계셔서 해당 부분 주석처리하였습니다.

[cycle-info-request.json]

  • 테스트가 필요한 경우를 고려하여 60개의 json데이터 삽입하였습니다.

🙏 아래 내용이 모두 충족 되었는지 확인해주세요 🙏

  • PR 이전 dev 브랜치 병합 하셨나요?
  • PR 이전 빌드 테스트 정상적으로 성공했나요?
  • PR 상세내용이 충분히 기재 되었나요?
  • PR 리뷰어, 할당자, 라벨, 프로젝트 확인

@Suxxxxhyun Suxxxxhyun added the 🐛 Fix 버그 수정 label Feb 8, 2025
@Suxxxxhyun Suxxxxhyun self-assigned this Feb 8, 2025
@Suxxxxhyun Suxxxxhyun changed the title fix: 주기정보저장시 차량정보의 위경도값저장하도록수정 fix: 주기정보저장시 차량정보의 위경도값 저장하도록 수정 Feb 8, 2025
Copy link
Collaborator

@kbyunghoon kbyunghoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰 남겼습니다! 확인해주세요

Copy link
Collaborator

@tomatozil tomatozil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

레디스 컨테이너 추가하신 것 좋네요👍
JPA 영속성 컨텍스트 변경 감지 활용하셨군요
고생하셨습니다

Copy link
Collaborator

@kbyunghoon kbyunghoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!

@Suxxxxhyun Suxxxxhyun merged commit da5f0f4 into develop Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Fix 버그 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants