Conversation
- 게시판 요청/서비스에서 이미지 파일 관리 지원 - 일반 파일과 이미지 파일 구분 - FileService에서 파일 업데이트/삭제 로직 중앙 집중화 - BOARD_IMAGE, TEMP 도메인 타입 추가 - S3Service에서 디렉터리 간 파일 이동 지원 - presigned URL 생성 시 domainType 제거, 기본 TEMP 디렉터리 사용
- FileService에 syncFiles 및 findFileById 메서드 추가 - 파일 도메인 업데이트 및 동기화 로직 중앙화 - 서비스 클래스의 직접 Repository 접근 제거 - 수동 파일 업데이트/삭제 로직을 FileService 메서드로 대체
Closed
1 task
wngktjd13
reviewed
Aug 18, 2025
wngktjd13
reviewed
Aug 18, 2025
wngktjd13
approved these changes
Aug 21, 2025
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.
#️⃣연관된 이슈
#79 #81
📝작업 내용
리팩터링: 파일 도메인 업데이트 및 동기화 로직
기능 추가: 게시판 이미지 파일 관리
게시판 요청 및 서비스 계층에서 이미지 파일 관리 기능 지원
일반 파일과 이미지 파일을 구분하여 처리 가능하도록 개선
파일 업데이트/삭제 로직을 FileService로 일원화
BOARD_IMAGE, TEMP 도메인 타입 추가
S3Service에 디렉터리 간 파일 이동 기능 추가
Presigned URL 생성 시 domainType 파라미터 제거, 기본 TEMP 디렉터리 사용으로 단순화
오전 4시에 DB 내의 임시 파일 데이터를 일괄 삭제하도록 구현하였습니다.
스크린샷 (선택)
💬리뷰 요구사항(선택)