Skip to content

Conversation

@Jinho622
Copy link
Contributor

@Jinho622 Jinho622 commented Aug 19, 2025

#️⃣연관된 이슈

#309

📝작업 내용

메인페이지 최근 문서 조회시 수정된 SNS 이벤트의 썸네일 이미지가 업데이트 되지 않는 버그 발생. SNS 이벤트 수정 시 최근 문서의 썸네일 이미지 키도 수정하여 해결
분위기 트레커의 제목 수정시에도 최근 문서의 썸네일 이미지 키가 수정되도록 추가 구현

🔎코드 설명(스크린샷(선택))

UserDocumentLastOpenedServiceImpl.java파일에 updateRecordsTitleAndThumbnailForWorkspaceUsers라는 LastOpened의 title과 썸네일 이미지 수정하는 메소드 생성하여 SnsEventCommandServiceImpl에서 사용
리포트가 생성될때도 적용시키기 위해 최근 문서의 썸네일 이미지 키를 업데이트하는 로직을 MoodTrackerReportServiceImpl의 uploadReportFileAndThumbnail메소드에 추가

💬고민사항 및 리뷰 요구사항 (Optional)

고민사항 및 의견 받고 싶은 부분 있으면 적어두기

비고 (Optional)

참고했던 링크 등 참고 사항을 적어주세요. 코드 리뷰하는 사람이 참고해야 하는 내용을 자유로운 형식으로 적을 수 있습니다.

@Jinho622 Jinho622 requested review from 2ghrms, djlim2425 and hknhj August 19, 2025 16:44
@Jinho622 Jinho622 self-assigned this Aug 19, 2025
@Jinho622 Jinho622 added the 🐞 Fix Something isn't working label Aug 19, 2025
@Jinho622 Jinho622 linked an issue Aug 19, 2025 that may be closed by this pull request
1 task
Copy link
Contributor

@hknhj hknhj left a comment

Choose a reason for hiding this comment

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

코멘트 확인 부탁드립니다!

// sns event 생성 시 워크스페이스에 속해있는 모든 유저에 대해
// last opened 테이블에 마지막으로 연 시간은 null로하여 추가
List<User> usersInWorkspace = userWorkspaceRepository.findUsersByWorkspaceId(savedSnsEvent.getWorkspace().getId());
userDocumentLastOpenedService.updateRecordsTitleAndThumbnailForWorkspaceUsers(usersInWorkspace, savedSnsEvent, request);
Copy link
Contributor

Choose a reason for hiding this comment

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

제목이 변하면 무조건 thumbnail key도 변하는 로직일까요?
만약 그렇다면 meeting, sns event, mood tracker 모두 그럴까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

맞습니다 mood tracker와 meeting에도 같은 메소드 추가시켜주어야 합니다! mood tracker는 현재 브랜치에서 수정해서 반영했고 meeting은 동재님이 해당기능 구현하실 때 추가해주시면 될 것 같습니다.

…bnail_key_name컬럼의 데이터 업데이트

리포트가 생성될때도 적용시키기 위해 MoodTrackerCommandServiceImpl.java파일의 updateTitle에 있던 로직을 MoodTrackerReportServiceImpl의 uploadReportFileAndThumbnail메소드에 추가
@Jinho622 Jinho622 merged commit 642a593 into dev Aug 21, 2025
1 check passed
@Jinho622 Jinho622 changed the title fix/#309: SNS 이벤트 수정 시 최근 문서의 썸네일 이미지 키도 수정 fix/#309: SNS 이벤트 수정 시 최근 문서의 썸네일 이미지 키도 수정 & 분위기 트레커의 제목이 수정시에도 최근 문서의 썸네일 이미지 키도 수정되도록 추가 구현 Aug 21, 2025
@Jinho622 Jinho622 changed the title fix/#309: SNS 이벤트 수정 시 최근 문서의 썸네일 이미지 키도 수정 & 분위기 트레커의 제목이 수정시에도 최근 문서의 썸네일 이미지 키도 수정되도록 추가 구현 fix/#309: SNS 이벤트 수정 시 최근 문서의 썸네일 이미지 키도 수정 & 분위기 트레커의 제목 수정시에도 최근 문서의 썸네일 이미지 키가 수정되도록 추가 구현 Aug 21, 2025
@Jinho622 Jinho622 changed the title fix/#309: SNS 이벤트 수정 시 최근 문서의 썸네일 이미지 키도 수정 & 분위기 트레커의 제목 수정시에도 최근 문서의 썸네일 이미지 키가 수정되도록 추가 구현 fix/#309: SNS 이벤트 수정 시 최근 문서의 썸네일 이미지 키도 수정 & 분위기 트레커의 제목 수정 시에도 최근 문서의 썸네일 이미지 키가 수정되도록 추가 구현 Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: SNS 이벤트 수정 시 최근 문서의 썸네일 이미지 키도 수정

5 participants