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

책 평가 리스트를 조회한다. #77

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

f1v3-dev
Copy link
Member

@f1v3-dev f1v3-dev commented Feb 7, 2025

연관된 이슈

resolves #76

작업 내용

  • 평가 Entity 및 JPA 세팅
  • 평가 리스트 조회 API 구성
  • Initializer 에서 평가 항목 추가 작업

리뷰 요구사항

확장성을 고려해서 EvaluationType, EvaluationKeyword을 구성했는데 괜찮을까요?

EvaluationType

  • 좋았어요: GOOD, 100번대 아이디 사용
  • 아쉬웠어요: SHAME, 200번대 아이디 사용

이렇게 구성한 이유는 새로운 키워드가 등장에 대해서 유연하게 대처하게 하도록 하고싶었어요.

ex. 별로에요: BAD 인 경우엔 300번대 아이디 사용

EvaluationKeyword

DB 테이블처럼 구조가 잡혀있는 형태에요!
만약에 MemberBookEntity 내부에 배열 형태의 컬럼(ex. [101, 103, 106, 204])이 존재한다면 변환하기 편할 것 같다고 생각이 들었어요.

@f1v3-dev f1v3-dev added the enhancement New feature or request label Feb 7, 2025
@f1v3-dev f1v3-dev requested a review from hwangdaesun February 7, 2025 06:28
@f1v3-dev f1v3-dev self-assigned this Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

책 평가 리스트를 조회한다.
1 participant