search

Project-MO-A / MOA-Backend

MO:A is a web application that provides group study or team project services
1 stars 2 forks source link

fix & refactor: 모집글 / 공지사항 api, 에러 수정 #87

Closed KAispread closed 1 year ago

KAispread commented 1 year ago

TO DO

KAispread commented 1 year ago

@Juhongseok 혹시 모집글 삭제랑 공지사항 조회 데이터 변경 작업 해주실 수 있을까요?? 시간 조율 API 수정하는데, PossibleTime 테이블 필드를 바꿔야되서 좀 걸릴 것 같아요

Discussion url 입니다 https://github.com/orgs/Project-MO-A/discussions/7

Juhongseok commented 1 year ago

네! 알겠습니다

Juhongseok commented 1 year ago

@KAispread 근데 테이블 필드가 많이 수정되나요?? LocalTime -> LocalDateTime으로 변경해주면 되지 않을까요???

KAispread commented 1 year ago

@KAispread 근데 테이블 필드가 많이 수정되나요?? LocalTime -> LocalDateTime으로 변경해주면 되지 않을까요???

요구사항대로 수정하면 Day, endTime 필드가 필요없어지고 LocalDateTime 하나만 남게 되는데 이러면 Dto랑 테스트코드를 다 수정해야됩니다ㅠ

Juhongseok commented 1 year ago

Day는 없어져도 endTime은 살린 상태에서 반환 값만 변경할 수 있지 않을까요?

applimentMemberId startTime endTime
1 2023-03-30T11:00:00.000Z 2023-03-30T14:00:00.000Z
1 2023-03-31T11:00:00.000Z 2023-03-30T14:00:00.000Z

이런 식으로 데이터 들어가면 반환 시에만 11시에서 14시까지 LocalDateTime.plusMinutes를 사용해서 30분 씩 더해서 값을 반환할 수 있을 거 같은데요?

그러면 response DTO 값 설정 부분만 수정하면 크게 바뀌진 않을 거 같아요

물론 LocalTime에서 LocalDateTime으로 변경하는 부분은 바뀌긴 하겠지만요..