Skip to content
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

이미지 파일 변경 시 기존 파일을 변경하는 로직을 추가하라 #89

Merged
merged 9 commits into from
Oct 31, 2023

Conversation

jihwooon
Copy link
Owner

@jihwooon jihwooon commented Oct 31, 2023

No description provided.

@jihwooon jihwooon changed the title 멀티 모듈에 file 정보를 검증하는 로직을 item 모듈에 추가하라 MulterOption을 ItemModule에서 분리하라 Oct 31, 2023
@jihwooon jihwooon changed the title MulterOption을 ItemModule에서 분리하라 이미지 파일 변경 시 기존 파일을 변경하는 로직을 추가하라 Oct 31, 2023
폴더 존재 여부에 따라 폴더를 생성하는 로직을 추가했습니다.
이미지 형식을 검증하는 로직도 추가했습니다
MulterModule에 구현되어 있던 로직들을 MulterOption 객체로 분리했습니다
UseInterceptors는 파일 업로드를 할 때 미리 해당 위치에 저장을 하는 역할을 합니다.
Multer 옵션을 추가해서 인터셉터를 하도록 진행했습니다
이미지 파일을 변경 시 기존 저장 된 이미지 파일을 대체하는 하도록 로직을 추가했습니다
상품 정보 변경와 이미지 파일 변경 하는 로직을 추가했습니다
변경 할 파일 정보와 item 정보을 전달 받으면 item의 Id 값으로 ItemImage 데이터베이스에서 저장 된 이미지를 가져옵니다.

가져온 이미지 URL를 이미지가 저정 된 폴더에서 삭제를 합니다. 그리고 새로운 정보를 데이터베이스에서 변경합니다.
이미지 파일을 변경 시 테이블에 이미지 정보를 변경하는 로직을
구현했습니다
ItemImageUpdater를 ItemModule에 추가했습니다
ItemImageUpdater 클래스 추가로 e2e 테스트들에 추가했습니다
@jihwooon jihwooon merged commit f50bc5b into main Oct 31, 2023
1 check passed
@jihwooon jihwooon deleted the issue-57 branch October 31, 2023 13:27
@jihwooon jihwooon added the Feture label Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant