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

data 하위 모델이 core.model을 의존하지 않도록 개선 #64

Merged
merged 7 commits into from
Jul 22, 2023
Merged

data 하위 모델이 core.model을 의존하지 않도록 개선 #64

merged 7 commits into from
Jul 22, 2023

Conversation

laco-dev
Copy link
Contributor

Issue

Overview (Required)

  • core.model은 순수한 모델 역할만 하기 위해 Serializable과 의존성을 가지지 않음
  • 이에 따라 관련 직렬화 로직을 Response로 분리

기타 작업사항

  • 서버에서 내려올 수 있는 값이 null인 경우 명확하게 nullable로 바꾸고 Mapper 레벨에서 기본 값을 지정하도록 함
    *Response가 non-null 타입이라면, 진짜 내려와야 하는데 안내려온건지 문제가 있는지 알 수 없음

@laco-dev laco-dev requested a review from wisemuji July 22, 2023 08:47
Copy link
Member

@wisemuji wisemuji left a comment

Choose a reason for hiding this comment

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

감사합니다!

@laco-dev laco-dev merged commit 9a39f46 into droidknights:main Jul 22, 2023
@laco-dev laco-dev deleted the feature/#63 branch July 22, 2023 10:34
@laco-dev laco-dev self-assigned this Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

data 하위 모델이 core.model을 의존하지 않도록 개선
3 participants