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

[#46] FIX : 댓글 오류 수정 #47

Merged
merged 3 commits into from
Aug 21, 2023
Merged

[#46] FIX : 댓글 오류 수정 #47

merged 3 commits into from
Aug 21, 2023

Conversation

gombasan
Copy link
Member

작업 사항

오류 내용

1. 댓글에 대한 생성날짜 수정날짜가 null 로 반환

KakaoTalk_Photo_2023-08-20-21-46-08

2. 대댓글을 등록 시 테이블에 내용이 정상적으로 등록이 되지 않음. (parentId = null)

KakaoTalk_Photo_2023-08-20-21-46-21

작업 내용

chore : BaseEntity 필드명 수정

  • createAt -> createDate 로 변경하였습니다.
  • DTO 객체와 서로 다른 필드명으로 인하여 코드량이 증가하여 통일하였습니다.
  • 기존 createAt 을 사용하는 클래스들도 같이 수정하였습니다.

chore : comment 데이터 매핑 수정

  • 게시글로 댓글을 내용과 함께 조회 할 경우 Mapper 가 정상적으로 날짜를 매핑하지 못하는 문제가 발생하였습니다.
  • BaseEntity 를 수정하여 정상적으로 데이터가 매핑되도록 수정하였습니다.
  • 답글을 등록 할 경우도 마찬가지로 변수명이 일치하지않아, 정상적으로 답글이 등록이 안되는 문제가 발생하여 수정하였습니다.
  • 댓글의 이모지도 함께 조회 하도록 추가하였습니다.

chore(Test) : comment 문서화 내용 변경

  • 공통 코드와 메시지를 분리하여 출력하도록 변경하였습니다.
  • 이모지에 대한 내용도 함께 문서화 하도록 수정하였습니다.

- `createAt` -> `createDate` 로 변경하였습니다.
- DTO 객체와 서로 다른 필드명으로 인하여 코드량이 증가하여 통일하였습니다.
- 기존 `createAt` 을 사용하는 클래스들도 같이 수정하였습니다.
- 게시글로 댓글을 내용과 함께 조회 할 경우 Mapper 가 정상적으로 날짜를 매핑하지 못하는 문제가 발생하였습니다.
- `BaseEntity` 를 수정하여 정상적으로 데이터가 매핑되도록 수정하였습니다.
- 답글을 등록 할 경우도 마찬가지로 변수명이 일치하지않아, 정상적으로 답글이 등록이 안되는 문제가 발생하여 수정하였습니다.
- 댓글의 이모지도 함께 조회 하도록 추가하였습니다.
- 공통 코드와 메시지를 분리하여 출력하도록 변경하였습니다.
- 이모지에 대한 내용도 함께 문서화 하도록 수정하였습니다.
@gombasan gombasan added 🐞 BugFix Something isn't working 📬 API 서버 API 통신 🔨 Refactor 코드 리팩토링 labels Aug 20, 2023
@gombasan gombasan self-assigned this Aug 20, 2023
@github-actions
Copy link

테스트 커버리지 리포트입니다

File Coverage [85.63%] 🍏
CommentResponse.java 100% 🍏
PostMapper.java 100% 🍏
HabitServiceImpl.java 99.37% 🍏
PostCustomRepositoryImpl.java 93.57% 🍏
HabitResponse.java 89.04% 🍏
DiaryResponse.java 89.01% 🍏
BaseEntity.java 81.25% 🍏
MyHabitResponse.java 77.59%
HabitCheckResponse.java 10%
Total Project Coverage 77%

@gombasan gombasan linked an issue Aug 21, 2023 that may be closed by this pull request
2 tasks
@gombasan
Copy link
Member Author

@JoosungKwon
머지하도록 하겠습니다~~~~~!!!

@gombasan gombasan merged commit bc49b1c into develop Aug 21, 2023
2 checks passed
@JoosungKwon
Copy link
Collaborator

@gombasan
미리 확인 못해서 죄송합니다. 고생하셨습니다~! 하지만, 중요한 점을 놓치신 거 같아요! 변경이 일어난다면 도메인코드보다는 DTO를 수정하는것이 변경에 여파가 적을 것 같습니다. DTO의 변화가 도메인에 영향을 주는 것은 좋지 못한거 같아요. 그리고 도메인 네임이 변경되었기 테이블도 수정해야합니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📬 API 서버 API 통신 🐞 BugFix Something isn't working 🔨 Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

댓글 api 오류 수정
2 participants