Skip to content

Chore: Image Proto 파일 수정#5

Merged
dungbik merged 1 commit intomainfrom
feat/update-profile
Mar 12, 2026
Merged

Chore: Image Proto 파일 수정#5
dungbik merged 1 commit intomainfrom
feat/update-profile

Conversation

@dungbik
Copy link
Contributor

@dungbik dungbik commented Mar 12, 2026

Summary by CodeRabbit

새로운 기능

  • 여러 이미지의 URL을 효율적으로 조회하는 대량 조회 기능이 추가되었습니다. 이제 여러 이미지 ID를 한 번에 전달하면 모든 이미지의 URL 정보를 빠르게 받을 수 있습니다.
  • 이미지 관리를 위한 삭제 기능이 추가되었습니다. 이미지 ID를 통해 필요 없는 이미지를 간편하게 삭제할 수 있습니다.
  • 이미지 활성화 시 응답 형식이 개선되어 URL 정보가 직접 포함됩니다.

@dungbik dungbik requested a review from stoneTiger0912 March 12, 2026 09:56
@dungbik dungbik self-assigned this Mar 12, 2026
@dungbik dungbik added the enhancement New feature or request label Mar 12, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 12, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 93402059-bf55-4112-adc1-54bec43168ab

📥 Commits

Reviewing files that changed from the base of the PR and between 0986884 and a5edf6b.

📒 Files selected for processing (1)
  • src/main/proto/image.proto

Walkthrough

이미지 서비스 API의 프로토콜 버퍼 정의가 업데이트되었습니다. 새로운 RPC 메서드 두 개(GetUrlsByIds, DeleteById)와 관련 메시지 타입들이 추가되었으며, ActivateImageResponse에 url 필드가 추가되었습니다.

Changes

Cohort / File(s) Summary
ImageCommandService API 확장
src/main/proto/image.proto
GetUrlsByIds 및 DeleteById RPC 메서드 추가. GetUrlsByIdsRequest, GetUrlsByIdsResponse, DeleteByIdRequest, DeleteByIdResponse 메시지 타입 새로 정의. ActivateImageResponse에 url 필드 추가.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • Feat: 이미지 변경 로직 간소화 #3: ActivateImageResponse에 추가된 url 필드를 활용하여 UserService에서 별도의 GetUrl RPC 호출 제거 - 동일한 이미지 서비스 응답 변경과 관련됨.

Suggested reviewers

  • stoneTiger0912

Poem

🐰 새로운 주소들을 한번에 얻고,
이미지들을 쓱 지워내고,
응답에 URL을 담아서,
API는 더욱 영리하게 변했네! ✨
프로토는 춤을 춘다네요!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/update-profile

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@dungbik dungbik merged commit 9e9c6dd into main Mar 12, 2026
2 of 3 checks passed
@dungbik dungbik deleted the feat/update-profile branch March 12, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant