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

refactor: 8/2 시점에서 merge후 개선사항 리팩토링 #244

Closed
10 tasks done
hyeon0208 opened this issue Aug 2, 2024 · 0 comments
Closed
10 tasks done

refactor: 8/2 시점에서 merge후 개선사항 리팩토링 #244

hyeon0208 opened this issue Aug 2, 2024 · 0 comments

Comments

@hyeon0208
Copy link
Contributor

hyeon0208 commented Aug 2, 2024

📝 작업 내용

  • feat: 나노초를 제외한 날짜와 시간을 데이터베이스에 저장 #229 해당 이슈 작업을 선행해 DB에 나노초가 제거 후 LocalDateTime이 저장되도록 수정 후 아래 작업을 하면 좋을듯
  • (나노초를 제거 후 저장 작업이 선행되면 내용 바뀜)s endAt이 나노초로 저장되기 때문에 notificationServiceTest에서 즉시 전송되는 알림을 조회 할 때 today도 withNano(0)으로 나노 초 제거 후 filter 조건에서 notification.getSendAt().isEqualTo(today) 로 비교하면 좋을 듯
  • develop 코드에 머지 시 입장 알림 타입과 출발 알림 타입의 구분을 위해 notificationServiceTest에서 즉시 전송되는 알림을 조회 할 때 filter 조건에 출발 알림 타입 필터링 구문도 필요 할듯
  • feat: 약속 참여자 ETA 조회 시, 수도권 외 지역 예외처리 #345
  • 약속 참여자 ETA 조회시, 행방불명 요청시 500에러 (위도 경도가 null일 때 계산이 안되서 오류 발생 추측)
  • meetingId가 중복될 수 없는 UUID로 변경
  • MateEtaResponse가 ARRIVED면 소요시간 0, MISSING이면 -1, ARRIVAL_SOON이면 1 응답되도록 수정.
  • polling 요청 받는 중간에 MateEtaRequest의 위도 경도가 잘못됐을 경우에 대한 예외도 처리 해야함.
  • 내 약속 목록 조회 시, 오늘 이후의 약속만 조회되도록 수정
  • MeetingController가 EtaService를 직접 호출하도록 코드 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

1 participant