Skip to content

Conversation

@koosco
Copy link
Member

@koosco koosco commented Jan 28, 2026

summary

  • 폴더 삭제 시 사진 제외 기능
  • 폴더에서 사진 제외 API 추가

details

폴더 삭제 시 사진 제외 기능

  • 폴더 삭제 시 포함된 사진을 삭제할지 여부에 따라
    1. 사진과 폴더를 함께 삭제
    2. 폴더만 삭제하고 사진은 유지 (연관관계만 제거)
    • requestParam을 이용해 삭제 여부를 전달합니다

폴더에서 사진 제외 API 추가

  • 폴더에서 사진을 제외시키는 API를 추가합니다
  • 폴더 내에서 사진을 삭제할 때 사진을 삭제하는 경우 DELETE /api/photos를, 폴더에서 제외만 시키는 경우 DELETE /api/folder/{folderId}/photos를 호출합니다

@koosco koosco self-assigned this Jan 28, 2026
@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Test ✅ PASSED

Test Result: success

✨ All tests passed!


Pushed by: @koosco, Action: pull_request

Base automatically changed from fix/#83 to staging January 28, 2026 14:50
@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Test ❌ FAILED

Test Result: failure

⚠️ Tests failed!

Please check the test output and fix the failing tests.

./gradlew test

Pushed by: @koosco, Action: pull_request

@koosco koosco requested a review from Darren4641 January 29, 2026 09:01
@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Test ❌ FAILED

Test Result: failure

⚠️ Tests failed!

Please check the test output and fix the failing tests.

./gradlew test

Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Test ✅ PASSED

Test Result: success

✨ All tests passed!


Pushed by: @koosco, Action: pull_request

Copy link
Contributor

@Darren4641 Darren4641 left a comment

Choose a reason for hiding this comment

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

Comment 1개 추가했습니다!

}

// 폴더에서 사진 없앰
photoImageRepository.updatePhotosFolderIdToNull(command.userId, command.folderIds)
Copy link
Contributor

Choose a reason for hiding this comment

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

사진까지 삭제하는 경우에도 이 update쿼리가 날라가더라구요 분기 태우면 쿼리 1개 덜 태울 수 있을 것 같아요!

@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Test ✅ PASSED

Test Result: success

✨ All tests passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Code Format Check ✅ PASSED

Spotless Check: success

✨ All code formatting checks passed!


Pushed by: @koosco, Action: pull_request

@github-actions
Copy link

Test ✅ PASSED

Test Result: success

✨ All tests passed!


Pushed by: @koosco, Action: pull_request

Copy link
Contributor

@Darren4641 Darren4641 left a comment

Choose a reason for hiding this comment

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

동작테스트했는데 잘 되네요 LGTM!

@Darren4641 Darren4641 merged commit 218d123 into staging Jan 29, 2026
2 checks passed
@Darren4641 Darren4641 deleted the feat/#84 branch January 29, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants