-
Notifications
You must be signed in to change notification settings - Fork 2
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
refactor: API 주소 변경 #67
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰가 늦어져서 죄송합니다. 개인적인사정으로 병원 치료때문에 정신이 없었네요 : (
전체적으로 API 명명을 변경하니 명시적으로 바뀐것 같아보이네요. 하나씩 코드부분 확인하면서 동작까지 확인하였습니다.
리뷰 확인 부탁드리겠습니다!
public void deletePhoto(TrashPhotoDTO.TrashPhotosRequest request, Long userId) { | ||
List<TrashPhoto> trashPhotos = trashPhotoRepository.findAllById(request.getTrashPhotoIds()); | ||
|
||
List<Long> storagePhotoIds = trashPhotos.stream() | ||
.map(TrashPhoto::getStoragePhotoId) | ||
.collect(Collectors.toList()); | ||
|
||
storagePhotoService.delete(userId, new StoragePhotoDTO.DeleteStoragePhotos(storagePhotoIds)); | ||
trashPhotoRepository.deleteAllByIdIn(request.getTrashPhotoIds()); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요청과 일치하는 휴지통 사진이 없습니다.
로 체크하는 로직이 사라진 이유가 혹시 있을까요? 만약 request로 들어온 List<Long> trashPhotoIds
id리스트가 없는 경우요!
resolve #66
여러 사진 휴지통으로 보내기
DELETE /storage-photos , body storagePhotoIds 배열
-> 200 OK, body에 trash-photos 배열
-> 대충 응답을 보고 아 휴지통으로 이동이 되는구나 할 수 있지 않을까, 단건이었으면 201 CREATED에 trash-photos location 헤더 를 넣어주는 식으로 표현 할 수 있겠다 라고 생각했습니다.
휴지통에서 여러 사진 복구
POST /trash-photos/restore, body trashPhotoIds 배열
-> 200 OK, body에 복구된 storage-photos 배열
휴지통 사진에서 사진 삭제 (보관함 사진 삭제)
DELETE /trash-photos, body trashPhotoIds 배열
-> 204