Skip to content

feat: 코스발견 탭 상단 배너 조회 API 추가 (prod) - #210

Merged
unam98 merged 2 commits into
mainfrom
feat/banner-api-prod
Aug 3, 2026
Merged

feat: 코스발견 탭 상단 배너 조회 API 추가 (prod)#210
unam98 merged 2 commits into
mainfrom
feat/banner-api-prod

Conversation

@unam98

@unam98 unam98 commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

배경

#209 (dev 반영 완료)의 배너 조회 API를 prod에도 반영. dev 브랜치 전체를 머지하면 배너와 무관한 대규모 인프라 변경(Grafana/Prometheus/ELK, deploy.sh 재작성 등, 아직 미검증)이 함께 나가므로, 배너 API 커밋만 별도로 cherry-pick해서 올림.

변경 사항

#209와 동일:

  • Banner 엔티티, BannerRepository, GetBannerResponseDto/BannerResponse, BannerService, BannerController
  • GET /api/banner — 활성 배너를 sortOrder(동률 시 id) 순으로 반환
  • SuccessStatus.GET_BANNER_SUCCESS 추가

영향 범위

신규 도메인 추가로 기존 API/로직에는 영향 없음. 인증 불필요한 공개 엔드포인트. main 대상 배포이므로 병합 시 prod EC2에 실제 배포됨.

검증

  • #209에서 로컬 docker-compose 기동 검증 + CodeRabbit 리뷰 2건 대응 완료
  • cherry-pick 후 main 기준으로도 ./gradlew compileJava 성공 확인

alh0409 added 2 commits August 3, 2026 15:53
Banner 엔티티/Repository/Service/Controller 추가, GET /api/banner로 활성 배너를 sortOrder 순으로 반환. Android 연동은 후속 작업.
sortOrder에 유니크 제약이 없어 값이 같은 배너가 있으면 정렬 순서가 요청마다 달라질 수 있어, id ASC를 보조 키로 붙여 결정론적으로 정렬되도록 함.
@unam98 unam98 self-assigned this Aug 3, 2026
@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 273b6ca1-fee4-4931-badf-6036b42346f0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@unam98
unam98 merged commit 24a1d8a into main Aug 3, 2026
2 checks passed
@unam98
unam98 deleted the feat/banner-api-prod branch August 3, 2026 06:54
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.

2 participants