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

[Session] 세션 data layer 구현 #62

Merged
merged 13 commits into from
Jul 22, 2023
Merged

Conversation

wisemuji
Copy link
Member

Issue

Overview (Required)

  • 세션 data layer 구현
  • :core:model 모듈 분리
    • 공통으로 사용하는 모델에 한해 별도의 모듈로 분리했습니다.
    • Session 모델은 원시값 타입 프로퍼티보다 별도의 파생 모델이 많이 생기는 구조입니다. 불필요한 모델 변환 코드를 전부 만들기보다는, 전 레이어에 공통적으로 사용될 개념이기에 공통 모델을 들고 있는 모듈이 있어야 한다는 필요성을 느꼈습니다.
    • NowInAndroid, DroidKnights 2021 구조를 참고했습니다.

Links

  • 세션 정보 데이터 저장소 참고사항: DroidKnights 2021 레포
  • 이전 PR에서의 공통 모델 레이어 분리 고민 코멘트: #32

Etc

  • Kotlinx LocalDateTime 의존성 추가
  • Kotlinx Serialization의 Enum 기본값 처리 위해 다음 옵션 추가
    • coerceInputValues = true

@wisemuji wisemuji added the enhancement New feature or request label Jul 21, 2023
@wisemuji wisemuji requested a review from laco-dev July 21, 2023 16:03
@wisemuji wisemuji self-assigned this Jul 21, 2023
@wisemuji wisemuji requested a review from laco-dev July 22, 2023 07:53
Copy link
Contributor

@laco-dev laco-dev left a comment

Choose a reason for hiding this comment

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

남겨드린 내용 하나만 확인해서 남겨주세요. 🙇🏻

@wisemuji wisemuji merged commit 4c52bee into droidknights:main Jul 22, 2023
@wisemuji wisemuji deleted the feature/#44 branch July 22, 2023 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2023 enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Session] 세션 data layer 구현
3 participants