Skip to content

refactor: 홈 화면 통합 API 제거 및 개별 API 분리 방식으로 전환#123

Merged
neibler merged 1 commit intodevelopfrom
feature/121
Feb 12, 2026
Merged

refactor: 홈 화면 통합 API 제거 및 개별 API 분리 방식으로 전환#123
neibler merged 1 commit intodevelopfrom
feature/121

Conversation

@neibler
Copy link
Copy Markdown
Contributor

@neibler neibler commented Feb 12, 2026

🔍️ 작업 내용

  • Closes #
    refactor: 홈 화면 통합 API 제거 및 개별 API 분리 방식으로 전환

✨ 상세 설명

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

📸 스크린샷 (선택)

💬 리뷰 요구사항

Summary by CodeRabbit

릴리스 노트

  • 변경사항
    • /api/home 엔드포인트가 제거되었습니다.
    • /api/advertisements 엔드포인트가 공개 접근 가능하도록 업데이트되었습니다.
    • 홈 데이터 수집 및 반환 기능이 제거되었습니다.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 12, 2026

Walkthrough

홈 API 엔드포인트 전체가 제거되었습니다. HomeController, HomeService, HomeConverter, HomeResDto 클래스를 삭제하고, SecurityConfig에서 /api/home 허가를 제거한 후 /api/advertisements/** 허가를 추가했습니다.

Changes

Cohort / File(s) Summary
Home API 제거
src/main/java/com/ongil/backend/domain/home/controller/HomeController.java, src/main/java/com/ongil/backend/domain/home/service/HomeService.java, src/main/java/com/ongil/backend/domain/home/converter/HomeConverter.java, src/main/java/com/ongil/backend/domain/home/dto/response/HomeResDto.java
GET /api/home 엔드포인트와 이를 지원하는 서비스, 변환 로직, DTO를 포함한 홈 데이터 집계 기능 전체 삭제.
보안 설정 변경
src/main/java/com/ongil/backend/global/config/SecurityConfig.java
/api/home의 공개 허가 규칙 제거 및 /api/advertisements/** 공개 허가 규칙 추가.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related issues

  • [Refactor] 홈 API 분할 #121: 홈 API 리팩토링 작업으로, 이 PR에서 HomeController, HomeService, HomeResDto 등의 제거를 통해 API 구조 개선을 진행하는 것과 관련됨.

Possibly related PRs

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 변경 사항의 주요 의도를 명확하게 설명합니다. 통합 API 제거 및 개별 API 분리 방식으로의 전환이라는 핵심 리팩토링 목표가 제목에 잘 반영되어 있으며, 실제 변경사항(HomeController, HomeService, HomeConverter, HomeResDto 제거 및 SecurityConfig 수정)과 일치합니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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/121

No actionable comments were generated in the recent 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 and usage tips.

@neibler neibler merged commit 7c6a812 into develop Feb 12, 2026
1 check passed
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