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

활동 상세 조회 API 응답 수정 완료 #459

Merged
merged 2 commits into from
Aug 14, 2024
Merged

Conversation

mingmingmon
Copy link
Collaborator

Summary

#455

활동 상세 조회 API의 응답을 ActivityGroupDetailResponseDto로 통일하였습니다. 해당 responseDto에는 ActivityGroup 도메인 엔티티의 모든 내용이 포함되어 있습니다.

Tasks

  • 활동 상세 조회 API의 응답 ActivityGroupDetailResponseDto로 통일하기

Screenshot

image

@mingmingmon mingmingmon added the 🔨 Refactor 코드 수정 및 개선 label Aug 12, 2024
@mingmingmon mingmingmon self-assigned this Aug 12, 2024
@mingmingmon mingmingmon requested a review from limehee as a code owner August 12, 2024 07:13
@limehee
Copy link
Collaborator

limehee commented Aug 12, 2024

해당 응답을 전달받는 모든 화면에서 ActivityGroupDetailResponseDto의 정보 모두를 필요로 하는 걸까요?
정보 모두를 필요로 하는 게 아니라면, DTO를 추가로 작성하여 필요한 만큼의 정보만 반환하는 게 좋다고 판단됩니다.

@mingmingmon
Copy link
Collaborator Author

해당 응답을 전달받는 모든 화면에서 ActivityGroupDetailResponseDto의 정보 모두를 필요로 하는 걸까요? 정보 모두를 필요로 하는 게 아니라면, DTO를 추가로 작성하여 필요한 만큼의 정보만 반환하는 게 좋다고 판단됩니다.

해당 작업을 진행하게 된 이유에 대해서 이슈에 적어놓았는데 정작 PR에는 설명이 부족했던 것 같네요.

프론트 @Jeong-Ag님의 요청으로 활동 상세조회 API에서 카테고리가 STUDY, PROJECT에 상관 없이 모든 정보가 반환될 수 있도록 수정하는 작업을 시작하게 되었습니다.

현재 프론트에서 활동을 생성할 경우 STUDY, PROJECT 구분 없이 imageUrl, curriculum, startDate, endDate, techStack, githubUrl을 선택입력으로 처리하고 있다고 합니다.

이전의 상세조회 API에서는 STUDY의 경우는 ActivityGroupStudyResponseDto에 imageUrl과 curriculum이 담기고, PROJECT에서는 ActivityGroupProjectResponseDtoimageUrl, startDate, endDate, techStack, githubUrl이 담겨 반환되고 있음을 설명드렸습니다만,

ActivityGroupDetailResponseDto로 통일하여 카테고리에 상관없이 모두 반환되도록 수정하시길 원해서 해당 작업이 진행되었습니다.

백엔드의 이전 API 설계에 따르면 ActivityGroupStudyResponseDtoActivityGroupProjectResponseDto를 따로 두고 있었습니다. 이로 미루어보아 프론트 측에서 활동 그룹을 생성할 때 카테고리를 구분해서 curriculum, startDate, endDate, techStack, githubUrl을 분리해서 받을 수 있도록 구현하는 것을 기대한 것 같습니다.

그러나 프론트측에서 활동 생성 화면에서 카테고리 구분 없이 해당 칼럼들을 선택 입력 처리를 하고 있다고 하였고, 상세조회 페이지에서는 관련 칼럼이 모두 반환되었으면 좋겠다고 요청하였습니다.

@limehee limehee merged commit f8c17da into develop Aug 14, 2024
3 checks passed
@limehee limehee deleted the refactor/#455 branch August 14, 2024 06:08
@limehee limehee linked an issue Aug 14, 2024 that may be closed by this pull request
1 task
@limehee limehee mentioned this pull request Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 수정 및 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

활동 상세 조회 API 응답 수정
3 participants