search

DDD-Community / undertheriver-sgsg-backend

✒ 사각사각 백엔드
9 stars 0 forks source link

fix: 최근 순으로 메모 조회 #148

Closed plzprayme closed 3 years ago

plzprayme commented 3 years ago

147

변경 사항

기타

hongbin-dev commented 3 years ago

  1. LocalDateTime을 그대로 프론트에 전달하고, 프론트에서 가공해서 사용하는 것으로 합의를 했었던 것 같은데 맞나요? 날짜 포맷말씀이신거죠? 협의안했던 것 같은데 이야기해보면 좋을 것 같아요.

  2. 인덱스를 추가해야 할 것 같습니다. 홍빈님께서 작업하신 feat: 인덱스 추가 및 네이밍 변경 코드를 수정 하겠습니다. 넵 알겠습니다!

  3. 지금 Memo에 @where이 없습니다. 소프트 딜리트 PR이 머지 된 devlop에 리베이스 후 머지해야 합니다. 넵!!

CI깨지는 것 같은데 확인한번 부탁드려요~

plzprayme commented 3 years ago

아 CI는 서브모듈 체크아웃할 때 credential 오류가 생기더라구요. 혹시 credential 다시 한번 확인해주실 수 있을까요?

hongbin-dev commented 3 years ago

149 이쪽 수정했습니다!!

plzprayme commented 3 years ago

그러고보니 저번 회의 때 날짜 포맷에 대한 논의를 안했군요 ㄷㄷ

plzprayme commented 3 years ago

일단은 서버에서 날짜포맷을 변경한 후 프론트로 보내도록 작업했습니다.

일단은 Memo 엔티티에서 LocalDateTime을 포맷팅하도록 작업했는데요.

createdAt을 VO로 감싸고 싶었는데 오류가 있었습니다. BaseEntity처럼 Auditing 되는 클래스에는 @Embedded 컬럼이 여러개 들어갈 수 없는지 제가 잘못한건지 잘 파악이 안되서 일단은 미뤄뒀습니다. 관련해서 작업 다 하고 트러블 슈팅 문서를 또 작성해봐야겠네요.

포기하지 않고 검색을 해서 다른 방법들을 찾아서 감쌀 예정입니다. 근데 작업 단위가 너무 커질 것 같아서 새로운 브랜치에서 작업해보겠습니다.

하여튼 createdAt, updatedAt을 VO로 감쌀 예정입니다...

혹은, 서버에서 포맷팅 하지 않고 프론트에서 포맷팅 하도록 요청해도 될 것 같구요.