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

경북대 BE_김동현 6주차 과제 (1단계) #114

Open
wants to merge 18 commits into
base: donghyuun
Choose a base branch
from

Conversation

donghyuun
Copy link

프론트엔드 분들과 협업하여 API 명세서를 작성하였습니다.
도메인명부터 필드 이름, 갯수, 응답 형식 등이 많이 달라 수정하는데 시간을 다소 사용했습니다.

개발 처음부터 프론트분들과 소통하여 API 명세서를 작성하는게 중요한 것 같습니다.

기타

  • DTO 에 "DTO" 문자열을 없앴더니 보기 편한 것 같습니다
  • DTO 에 toEntity, Entity 에 toDTO 메서드를 만드니 편한 것 같습니다.
  • DTO(record) 에 Inner record를 만드니 편한 것 같습니다.

아직 배포를 안해서 스웨거 문서 이미지 남깁니다!
image
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant