Skip to content

fix: 추천brand 수정#157

Merged
neibler merged 1 commit intodevelopfrom
feature/151
Feb 19, 2026
Merged

fix: 추천brand 수정#157
neibler merged 1 commit intodevelopfrom
feature/151

Conversation

@neibler
Copy link
Copy Markdown
Contributor

@neibler neibler commented Feb 19, 2026

🔍️ 작업 내용

  • Closes #
    fix: 추천brand 수정

✨ 상세 설명

🛠️ 추후 리팩토링 및 고도화 계획

📸 스크린샷 (선택)

💬 리뷰 요구사항

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 랜덤 브랜드 조회 시 상품과 브랜드 간의 정확한 연결 관계를 보장하도록 데이터베이스 쿼리 조건을 수정했습니다. 이제 브랜드 데이터가 올바르게 조회됩니다.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 19, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

BrandRepositoryfindRandomBrands 메서드 내 EXISTS 서브쿼리의 조인 조건을 수정했습니다. p.brand_id = b.id에서 p.brand_id = b.brand_id로 변경하여 상품과 브랜드 간 올바른 상관관계를 보장합니다.

Changes

Cohort / File(s) Summary
SQL 조인 조건 수정
src/main/java/com/ongil/backend/domain/brand/repository/BrandRepository.java
EXISTS 서브쿼리의 조인 조건 수정: p.brand_id = b.idp.brand_id = b.brand_id로 변경하여 상품-브랜드 상관관계 오류 해결

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/151

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.

@neibler neibler merged commit c276e6e into develop Feb 19, 2026
1 check was pending
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.

1 participant