Conversation
sae2say
left a comment
sae2say
left a comment
There was a problem hiding this comment.
역시 잘해주셨네요!! 너무 고맙숩니다
아직 디자인이 확정이 아니라고 하고, 필드 length 같은 부분도 통일해야하니 반영할 것 하고 늦게 머지해도 좋을 것 같아요! 수고하셨습니다!
| public enum MissionImageType { | ||
|
|
||
| MAIN, | ||
| EXAMPLE, | ||
| ; | ||
| } |
There was a problem hiding this comment.
미션 하나 당 하나의 대표이미지만 존재하는거라고 보여지는데 이 enum을 작성하신 이유가 있나요?
There was a problem hiding this comment.
와웅.... 와이어 프레임 보고 제가 착각했어요.... 해당 enum 및 필드 삭제해두겠습니다!
| @@ -0,0 +1,9 @@ | |||
| package com.JJIN.domain.mission.entity.enums; | |||
|
|
|||
| public enum MissionStatus { | |||
There was a problem hiding this comment.
네 맞습니다. 기능이 정확히 확정되어야하겠지만, 사용자가 만든 미션에 대해서 다른 사용자가 이미 미션 인증을 했다면 삭제 처리가 좀 복잡해질 것 같아 소프트딜리트 처리로 우선 해두었습니다!
| @@ -0,0 +1,8 @@ | |||
| package com.JJIN.domain.mission.entity.enums; | |||
|
|
|||
| public enum MissionSourceType { | |||
There was a problem hiding this comment.
미션 추천 탭에서 선택가능한 필터 세 개 중 (꼭 해봐야 할, 요즘 핫한, 내가 만든) '요즘 핫한'은 계산으로 도출되는 부분이니 따로 enum을 추가하지 않으신 건가요?
저는 한 3-4일 정도 기준을 두고, 그 기간동안 미션들의 수치를 집계해서 '요즘 핫한' 미션을 도출해서, 그 3-4일 동안에는 '요즘 핫한' 미션이 바뀌지 않는 거라고 생각했었는데
윤아님은 미션 추천 탭이 조회될 때마다 계산 알고리즘이 수행되게 하여 '실시간 요즘 핫한' 을 생각하신건가요?
There was a problem hiding this comment.
저는 '실시간 요즘 핫한 게시물'을 생각했어서 계산으로 도출되는 부분이라고 생각했습니다! 다만 세희님 의견처럼 매번 실시간으로 집계하는 것 보다는 3_4일 간격으로 계산해서 하는게 더 효율적일 것 같네요!! 다른 팀원분들과도 논의 후에 확정 하면 수정해두겠습니다!
| @Column(length = 50) | ||
| private String category; |
There was a problem hiding this comment.
화면 상에서 카테고리를 입력받는 부분은 없고, 카테고리를 입력받는다고 해도 신경쓸 게 많을 것 같네요. 제 개인적인 생각인데 , 사용자가 직접 카테고리를 입력하는 게 아닌, 장소나 제목, 내용 등을 입력하면 LLM이 미리 정해둔 n개의 카테고리 중 분류하여 자동으로 저장되도록 하면 좋을 것 같네요. 어떻게 생각하세요?
There was a problem hiding this comment.
현재 와이어 프레임 상으로는 미션 생성 시에는 카테고리가 아닌 해시태그로 좀더 자유롭게 사용자가 입력하고 있는 것 같아요!
세희님 의견처럼 미리 카테고리 개수를 정해 enum 처리하고 태그를 바탕으로 llm이 자동으로 분류할 수 있게 하는 것도 좋은 것 같습니다! 카테고리 종류는 팀원분들과 이야기 나누어보면 좋을 것 같아요!
|
|
||
| @ManyToOne(fetch = FetchType.LAZY) | ||
| @JoinColumn(name = "created_by_member_id", nullable = false) | ||
| private Member createdBy; |
There was a problem hiding this comment.
저희가 제공하는 기능을 고려했을 때 memberId보다 이게 더 명시적이네용 센스 굿
2 participants
Related issue 🛠
Work Description ✏️
MissionMissionImageMissionTagMissionTagMappingUserMissionUserMissionStatusMissionProofMissionProofImageMissionDifficultyMissionSourceTypeMissionStatusMissionImageTypeMemberMissionMissionTagMissionProofTrouble Shooting ⚽️
Store,Schedule,Trip,Plan엔티티가 없어 이번 PR에서는 관련 연관관계를 추가하지 않았습니다.요즘 핫한 미션은 아직 인기 산정 기준이 확정되지 않아Mission.hot필드로 두지 않았습니다. 추후 기준 확정 시MissionStats같은 별도 집계 모델로 분리할 예정입니다.Related ScreenShot 📷
Uncompleted Tasks 😅
To Reviewers 📢