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

[refact] 회고 기능 response dto 필드 정리 #184

Merged
merged 2 commits into from
Aug 13, 2024
Merged

Conversation

letskuku
Copy link
Member

@letskuku letskuku commented Aug 12, 2024

💡 PR Summary - 회고 기능 response dto 필드 정리

📝 Description


회고록 목록 보기 api와 회고록 상세 보기/수정 api에 사용되는 response dto의 필드에 다소 아쉬운 부분이 있어 수정 진행

  1. 회고록 목록 보기 api response dto 필드 수정 (제목, url, 작성자, 활동기수 -> 제목, 활동기수, 파트)
  2. 회고록 상세 보기/수정 api response 필드 수정 (제목, url, 작성자 -> 제목, url, 작성자, 활동기수, 파트)

-> 회고록 작성 api에서는 입력하지만 사용되지 않았던 파트를 dto에 추가하고 "상세 보기"라는 api 이름에 맞게 상세 보기 api에서 더 자세한 회고록 정보를 확인할 수 있도록 수정

세오스 프로젝트 페이지를 보면 목록 보기에서는 프로젝트 이름, 기수, 한줄소개를 보여주고 있어 회고록 목록 보기 dto도 비슷한 느낌으로 수정하였습니다.

🌲 Working Branch


refact/retrospect

📖 Related Issues


Copy link
Member

@yoonsseo yoonsseo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아아 같은 프로젝트를 작업한 사람들의 회고록을 모아놓는 방식이군용
수고하셨습니다👍🏻👍🏻

@letskuku
Copy link
Member Author

letskuku commented Aug 13, 2024

앗 그냥 회고록 목록 보기 dto 필드를 정하는 과정에서 프로젝트 목록 보기를 참고했다는 의미였습니다...!
그런데 말씀해주신 아이디어도 괜찮은 것 같네요👍

@letskuku letskuku merged commit b01f13c into dev Aug 13, 2024
@letskuku letskuku deleted the refact/retrospect branch August 13, 2024 09:21
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.

2 participants