Skip to content

[Fix] 전세대출필터링 조건 수정#114

Merged
bagjuna merged 26 commits intomainfrom
feature/financial_jeonseLoan
Aug 12, 2025
Merged

[Fix] 전세대출필터링 조건 수정#114
bagjuna merged 26 commits intomainfrom
feature/financial_jeonseLoan

Conversation

@khozan0120
Copy link
Copy Markdown
Member

📌 관련 이슈

Close #113

📄 PR 상세 내용

  • 전세대출 상품 추천 API에서 minAmount, maxAmount 파라미터 조건이 잘못 구현되어 있었음
  • 사용자가 설정한 대출 금액 범위 내에 완전히 포함되는 상품이 조회되지 않는 문제

📸 이미지 (테스트 이미지 필수)

필터링조건에서 minAmount 가 0이고 maxAmount가 2억원일때 전체조회
min이 0일때
필터링조건에서 minAmount 가 500만원이고 maxAmount가 2억원일때 전체조회
min이 500만원

❓ 논의하고 싶은 내용

📍 레퍼런스

…o feature/financial_jeonseLoan

# Conflicts:
#	src/main/java/org/scoula/domain/financialproduct/constants/JeonseLoanRateType.java
#	src/main/java/org/scoula/domain/financialproduct/jeonseloan/service/JeonseLoanServiceImpl.java
…o feature/financial_jeonseLoan

# Conflicts:
#	src/main/java/org/scoula/domain/financialproduct/personalcreditloan/dto/request/PersonalLoanSearchRequest.java
@khozan0120 khozan0120 self-assigned this Aug 12, 2025
@khozan0120 khozan0120 added the fix 수정 label Aug 12, 2025
Copy link
Copy Markdown
Collaborator

@bagjuna bagjuna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋습니다. 저도 이거 조금 햇갈리더라고요

@bagjuna bagjuna merged commit cd2ee52 into main Aug 12, 2025
@bagjuna bagjuna deleted the feature/financial_jeonseLoan branch August 12, 2025 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants