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

JPA 엔티티를 구현한다. #170

Merged
merged 7 commits into from
Jan 20, 2025
Merged

JPA 엔티티를 구현한다. #170

merged 7 commits into from
Jan 20, 2025

Conversation

GIVEN53
Copy link
Member

@GIVEN53 GIVEN53 commented Jan 20, 2025

  • close JPA 엔티티를 구현한다. #169
  • CommentLike 엔티티의 type 컬럼은 기획의 구체화가 필요하므로 본 PR에서 제외합니다.
  • 도메인 검증은 러프하게 짜두었습니다. 필요한 부분은 기능 개발하면서 추가해 나가면 될 것 같습니다.
  • 매일위키는 매일메일과 달리 유저로부터 컨텐츠가 생산되기 때문에 검증을 보다 빡빡하게 가져가야 하지 않을까 싶습니다.

Copy link

Test Results

60 tests   60 ✅  2s ⏱️
23 suites   0 💤
23 files     0 ❌

Results for commit 6aaaf53.

@GIVEN53 GIVEN53 merged commit c4d0579 into main Jan 20, 2025
3 checks passed
Copy link
Collaborator

@kunsanglee kunsanglee left a comment

Choose a reason for hiding this comment

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

👍❤️

return Arrays.stream(WikiCategory.values())
.filter(it -> it.name().equalsIgnoreCase(category))
.findFirst()
.orElseThrow(IllegalArgumentException::new);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Provider에서는 NoSuchElementException 던졌는데 통일되면 좋을 것 같아요 👍

@le2sky le2sky deleted the 169 branch January 30, 2025 11:41
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.

JPA 엔티티를 구현한다.
2 participants