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] #8 뉴스레터 조회 및 홈에 필요한 엔티티 설계 #9

Merged
merged 14 commits into from
May 12, 2024

Conversation

packdev937
Copy link
Contributor

@packdev937 packdev937 commented May 11, 2024

Description

홈에 필요한 전반적인 엔티티를 설계하였습니다. (단어장, 아카이브)

Issue Number

#8

Extra

User 엔티티와 Archive 엔티티는 단방향으로 설정하였습니다. Archive에서 User로 직접 조회할 필요가 없기 때문입니다.
추가로 Archive에서 가지고 있는 List termIds는 Converter를 통해 데이터베이스에 String 형태로 저장되도록 변경하였습니다.

추가로 LazyInitializationException이 발생함에 따라 @transactional 어노테이션을 활용하였습니다.

PR Type

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

@packdev937 packdev937 linked an issue May 11, 2024 that may be closed by this pull request
6 tasks
@packdev937 packdev937 merged commit 6811f61 into main May 12, 2024
1 check passed
@packdev937 packdev937 deleted the feature/8-home-api branch May 12, 2024 04:29
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.

[Feature] 뉴스레터 조회 및 홈에 필요한 엔티티 설계
1 participant