Conversation
laura-jung
approved these changes
Aug 18, 2026
laura-jung
left a comment
Contributor
There was a problem hiding this comment.
presignedUrl부터 AI 연결까지... 역시 대세희네요...
꼼꼼하게 작업해주셔서 감사합니다. 항상 많이 배워갑니다....
AI 때문에 변경된 yml도 있을 것 같아서 노션에 공유 부탁드릴게요!
코멘트 남긴것도 확인부탁드립니다!
Comment on lines
+87
to
+93
| if (source == MissionSourceTypeOption.HOT) { | ||
| return buildHotMissionsResponse(memberId); | ||
| } | ||
| if (source == MissionSourceTypeOption.ADDED) { | ||
| return buildAddedMissionsResponse(memberId); | ||
| } | ||
|
|
Contributor
There was a problem hiding this comment.
HOT이나 ADDED인 경우 여기 if 문에서 바로 반환되어 keyword,sort, page, size와 같은 다른 파라미터들이 적용되지 않을 것 같습니다.
source도 기존 검색 조건과 조합해서 필터링·페이징해야 다른 조건들도 함께 적용될 것 같아요! 아니면 의도적으로 반영되지 않게 기획하신 건지 궁금합니다!
Collaborator
Author
There was a problem hiding this comment.
HOT 미션은 계산 로직에서 top 5개로 자르고 있길래 바로 반환했는데, ADDED는 제가 놓쳤네요! ADDED에 적용해두겠습니다!
Comment on lines
+235
to
+236
| List<TravelPlan> plans = travelPlanRepository.findByMemberIdOrderByCreatedAtDesc(memberId); | ||
|
|
Contributor
There was a problem hiding this comment.
이곳에서도 미션 존재 여부를 확인하는 로직이 필요할 것 같아요! 미션이 삭제되어 비활성화된 missionId도 모든 일정이 isLiked=false인 정상 응답을 반환할 것 같습니다!
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
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.
Related issue 🛠
Work Description ✏️
미션 이미지 presigned URL 발급 API 구현
미션 생성 API 구현
미션 찜 추가 / 해제 API 구현
미션-일정 데이터 구조 설계
Mission 엔티티 정리
CI 개선
일정별 미션 찜 여부 조회 API 구현
미션 검색 피드에 소스 필터 추가
Trouble Shooting ⚽️
Clock이 UTC 기준으로 동작해 핫 미션 집계 윈도우가 UTC로 계산되고, addedAt은 JVM 기본 타임존(KST)으로 저장되어 타임존이 혼용되는 버그 수정. Clock을 Asia/Seoul 기준으로 변경해 집계 윈도우와 저장 시각의 타임존을 일치시킴.
Related ScreenShot 📷
presigned-url 발급 API
미션 생성 API
미션 찜 설정 API
미션 찜 해제 API
Uncompleted Tasks 😅
To Reviewers 📢
openAI 사용방식
제가 학교에서 종설할 때 지원받았던
LLM 호출 서드파티 서비스runyourai의 토큰이 꽤 남아있어서, 이걸 사용해서 LLM 호출하려구요! (18만원 정도 남아있습니다.)나중에 일정 추천 관련해서도 호출 막 하다보면 어느정도로 토큰이 소비될 진 잘 모르겠는데, 부족하면 그 때 가서 openai 다시 설정할게요!
마이그레이션
userMission의 제약이랑 인덱스를 수정해서, 로컬에서 마이그레이션이 필요할 수 있어요! userMission 테이블 드롭하고 다시 애플리케이션 실행하시거나, 마이그레이션 코드 첨부할게요! 그냥 이 쿼리 실행하면 됩니다