Skip to content

Conversation

@doup2001
Copy link
Member

@doup2001 doup2001 commented Jan 19, 2026

📌 작업한 내용

주변환경(주변 시설) 데이터 모델과 API를 새로 추가하여 부동산 검색 시 주변 편의시설 정보를 제공할 수 있도록 했습니다. 데이터베이스 테이블 생성, 엔티티/리포지토리 구현, REST API 엔드포인트 개발을 완료했습니다.

🔍 참고 사항

  • 주변환경: 학교, 병원, 지하철, 마트 등 카테고리별 데이터 구조화
  • 연동: 부동산 상세 정보에 주변환경 정보 포함
  • 성능: 공간 인덱스 적용으로 쿼리 최적화

🖼️ 스크린샷

UI 변경 사항 없음 (백엔드 기능)

🔗 관련 이슈

주변환경 검색 기능 개발 이슈

✅ 체크리스트

  • DB 테이블 및 인덱스 생성 완료
  • API 테스트 (Postman 등) 완료
  • 위치 기반 쿼리 성능 검증
  • 문서화 업데이트

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

Walkthrough

FacilityType 열거형에 6개의 새로운 상수(ELDER, CHILD, BICYCLE, DISABLED, YOUTH, POLICE)를 추가하고, NoticeFacilityListResponse와 FacilityService의 map 연산을 displayType() 문자열에서 실제 FacilityType 키로 변경하여 타입 일관성을 개선했습니다.

Changes

Cohort / File(s) 변경 요약
Enum 상수 확장
src/main/java/com/pinHouse/server/platform/housing/facility/domain/entity/FacilityType.java
ELDER, CHILD, BICYCLE, DISABLED, YOUTH, POLICE 6개의 새로운 enum 상수 추가
타입 맵핑 수정
src/main/java/com/pinHouse/server/platform/housing/facility/application/dto/NoticeFacilityListResponse.java,
src/main/java/com/pinHouse/server/platform/housing/facility/application/service/FacilityService.java
map 연산에서 entry.getKey().displayType() (String)에서 Map.Entry::getKey (FacilityType)로 변경하여 반환 타입을 String에서 FacilityType으로 수정

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 여섯 개의 새로운 시설이 들어섰네,
어르신, 아이, 자전거, 장애인 친구들,
청년, 경찰까지 함께 모여,
타입도 반듯하게 정렬되었지!
우리 마을이 더욱 풍요로워졌네! 🏘️

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명이 템플릿 구조는 포함하지만, 모든 필수 섹션이 비워져 있어 구체적인 변경 내용, 참고 사항, 관련 이슈, 체크리스트 확인이 없습니다. "작업한 내용" 섹션에 6개의 새로운 FacilityType 열거형 추가, 매핑 로직 수정 등 구체적인 변경 사항을 작성해주세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 새로운 주변환경(FacilityType) 추가를 명확하게 반영하며, 변경 사항의 핵심을 간결하게 요약합니다.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

@github-actions
Copy link

Test Results

22 tests  ±0   22 ✅ ±0   0s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 3 files   ±0    0 ❌ ±0 

Results for commit 6fac12a. ± Comparison against base commit 6fb85c3.

@doup2001 doup2001 merged commit 7dc4aa4 into main Jan 19, 2026
3 checks passed
@doup2001 doup2001 deleted the feat/인프라-관련-기능-구현 branch January 19, 2026 09:49
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