#11 - 애그리거트 설계에 맞춰 도메인 추가 #12
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sectionr0
reviewed
Jul 6, 2024
|
|
||
| data class Report( | ||
| val type: ReportType, | ||
| val reporter: User, |
There was a problem hiding this comment.
targetId 도 추가해주세요! message나 vote의 id 값도 있어야 신고 받았을 떄 추적하기 더 쉬울 것 같아요!
There was a problem hiding this comment.
아 그 타겟 id 였군요~! 추가하도록 하겠습니당
| val content: String, | ||
| val isRead: Boolean, | ||
| val readAt: LocalDateTime, | ||
| val isEnabled: Boolean, |
There was a problem hiding this comment.
이거는 새로 추가된 알림 비활성화에요! 사용자가 클릭 못하게 하는 것!
| @@ -0,0 +1,4 @@ | |||
| package com.wespot.school | |||
|
|
|||
| enum class EstType { // TODO : 이 부분은 같이 이야기하면서 구체화 해나가 보시죠! | |||
There was a problem hiding this comment.
EstType이 국립인지 공립, 사립 구분이여서 이 내용 도메인에서 제거하는거 어떠신가요?
| @@ -0,0 +1,4 @@ | |||
| package com.wespot.school | |||
|
|
|||
| enum class SchoolType { // TODO : 이 부분은 같이 이야기하면서 구체화 해나가 보시죠! | |||
There was a problem hiding this comment.
EstType, SchoolType 둘 다 wespotERD 따라서 진행하기는 했는데, 그래서 저도 감이 안왔거든요.
그래서 말씀하신대로 지우는게 좋을 것 같아요
| val id: Long, | ||
| val school: School, | ||
| val grade: Int, | ||
| val group: Int, |
There was a problem hiding this comment.
group 이 반을 의미를 하는거죠? class는 어떠신가요?!
There was a problem hiding this comment.
class 하려다가... class는 키워드라서..
sectionr0
approved these changes
Jul 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1. 🔗 관련 이슈
#11
2. 📄 구현한 내용 또는 수정한 내용
이슈에 나와있는 것과 동일한 구조로, 정말 간단하게 도메인 모델들만 추가했어요! 추가적으로 VO들 같은 경우는 각자 작업하면서 구체화 해나갈 것 같아서 딱히 크게 건드리지는 않았습니다!
3. 🙌 추가적으로 알리고 싶은 내용
Entity를 추가한다거나 구조를 잡는 것은 기천짱이 한번 보시고 제가 한번에 진행해도 되겠다 싶으면 코멘트 남겨주세요! 혹은 같이 페어 프로그래밍 해도 좋구요 ~~ 🥰
4. 🙄 TODO / 고민하고 있는 것들
5. ✅ 배포 Checklist