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

Feature/133 : Scrap API 응답 변경 #136

Merged
merged 7 commits into from
Jan 1, 2024
Merged

Feature/133 : Scrap API 응답 변경 #136

merged 7 commits into from
Jan 1, 2024

Conversation

swa07016
Copy link
Member

@swa07016 swa07016 commented Jan 1, 2024

🚀 개요

Scrap V2 등록, 삭제 API에 스크랩 개수와 스크랩 여부 필드를 추가했습니다.

⏳ 작업 내용

  • V1, V2 Api 클래스 분리
  • Scrap 커맨드 전략 V1, V2 구현
  • 스크랩 개수, 스크랩 여부 필드 추가

📝 논의사항

  • 초기에 정한 컨벤션을 유지한 채 서비스 계층에서 엔티티를 반환하다보니 아래와 같은 문제를 마주했습니다.
    1. 엔티티 정보 이외에 추가정보가 필요할 때, DTO extends Entity를 생성 (안티 패턴입니다..😅)
      • 엔티티에 Transient 필드로 추가 정보(ex. 통계값, 임시 필드)를 둘 수도 있지만 많아지면 엔티티가 DTO의 역할을 침범하는 꼴이므로 한계가 있습니다.
    2. 컨트롤러단에서 컨버팅 로직에 태울 때 Entity to DTO 다운캐스팅
  • 따라서, 설계를 다시 점검해볼 필요가 있을 것 같습니다.
  • 우선 이번 PR에서는 현재 방식을 유지하고 다음 이슈로 가져가서 개선하겠습니다!

@swa07016 swa07016 added ✨ feature New feature or request ♻️ refactor labels Jan 1, 2024
@swa07016 swa07016 requested a review from CYY1007 January 1, 2024 14:34
@swa07016 swa07016 self-assigned this Jan 1, 2024
@swa07016 swa07016 linked an issue Jan 1, 2024 that may be closed by this pull request
1 task
@swa07016 swa07016 merged commit 8b7e018 into develop Jan 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor ✨ feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🚀 [feature] 스크랩 API 응답 변경
1 participant