-
Notifications
You must be signed in to change notification settings - Fork 1
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
Impl Entity API (MEE013~027) #1298
base: dev
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
전체적으로 보이는 건
- 모듈 구조를 entity로 묶고 repository를 분리하기
- patch랑 put을 저는 반대 의미 (put이 데이터 밀어넣기)로 알고 있는데, 여긴 다 patch로 되어 있어서 put으로 수정하기
- patch든 put이든 httpstatus created로 되어 있는 부분들 ok로 변경하기
- 18,19,20 번 url 같은거 뒤에 엔티티 더 붙여서 수정하기
- 17번도 url 마지막에 result 붙이면 더 좋을 듯?
양 참 많은데 고생 많았어요~~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
meetingAgendaEntityType은 Enum이 아닌가? enum이라면 enum 테이블과 네이티브 이넘으로 만들면 좋을 것 같아요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테이블에 안만들기로 한건가? 그래도 native enum은 있어야 할 것 같은데...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
status created 수정 필요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그리고 이넘 테이블은 아는 바가 없어요.. 설계 당시에는 그냥 이넘만 쓰기로 했습니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지금 repository 를 entity에서 공유하고, 나머지 vote랑 content를 서브모듈로 둔 것 같은데, 구조가 반대가 되어야 하지 않을까 싶습니다. Entity 모듈, 서비스, 컨트롤러를 쓰면서 각 서브 엔티티(vote, content) 별로 repository를 두는 방식으로요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Entity 모듈, 서비스, 컨트롤러를 쓰면서 각 서브 엔티티(vote, content) 별로 repository를 두는 방식으로요
이거의 경우에는 vote, content 모듈을 아예 없애라는 말씀이신가요?
아니면 vote content 모두 모듈 서비스 컨트롤러에 repository까지 두라는 말씀이신가요
요약 *
It closes #1297
MEE013 - 027 구현 완료했습니다
테스트는 아직 못 해봤어요..! 26일까지 해보겠습니다!
Update 01/27
MEE024-MEE026을 그냥 Vote에 대해 모든 정보를 가져오는 API (24번)하나로 수정했습니다.그에 따라 비는 번호들은 그냥 비어 있도록 놔두었어요
13번부터 27번까지 모두 테스트 완료했습니다!
13: Content 생성, Mapping 생성
14: Content 내용 수정
15: Content 삭제, Mapping 삭제 (모두 soft delete)
16: Vote 생성, 매핑 생성, 선택지 생성
17: Vote에 대한 유저의 투표 내역 생성
18: Vote의 제목, 설명 수정
19: Vote의 선택지 수정 (선택지 항목 개수가 늘어날 수도 있어서, 모두 삭제 후 재생성하는 로직)
20: Vote에 대한 유저의 투표 내역 수정
21: Entities 순서 변경
22: Vote 삭제, 매핑 삭제, 선택지 모두 삭제, 유저의 투표 내역들 삭제 (모두 soft delete)
23: Vote에 대한 유저의 투표 내역 삭제
24: Vote에 대한 모든 정보 가져오기 (제목, 설명, 선택지들, 유저가 선택한 것, 최종결과 (표 집계내역))
27: 특정 미팅 특정 아젠다의 모든 엔티티 가져오기
스크린샷
이후 Task *