Repository navigation
Conversation
PNU-2026-AI-Hackathon/pnuai-c-07-finfin2#42 (33a2b2c)의 backend/, database/ 변경을 Fin-BE 경로로 옮겼다. - backend/ → 루트, database/db/migration → src/main/resources/db/migration - 충돌 해결: SearchRequestPolicy import (#33의 미사용 List 대신 Set) - 모노레포 전용 변경(.gitignore, api/)은 제외
#32에서 예금 한도가 min/max_deposit_amount로 분리되었는데 PRD A1~A3 이관분은 min_monthly_limit만 비교하고 있었다. - 예금: 예치 원금(단기 예치액, 목돈 월저축액×기간) vs min_deposit_amount - 적금: 월 납입액(단기 예치액÷기간, 목돈 월저축액) vs min_monthly_limit - 파킹: 예치액 vs min_deposit_amount (V8 기준 예금·파킹 공용)
Backend Code Health ReportCommit:
Attention
Code Issues
|
Collaborator
Author
|
#41로 대체합니다. V7 마이그레이션이 #35의 V8보다 늦게 들어와 기존 개발 DB에서 Flyway out-of-order로 기동에 실패해, V9로 옮겨 새 브랜치로 다시 올렸습니다. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
변경점 👍
해커톤 모노레포에서 머지된 PRD A1~A3 작업(PNU-2026-AI-Hackathon/pnuai-c-07-finfin2#42)을 Fin-BE로 옮겼습니다.
27c0717: 원본 PR의backend/,database/변경을 Fin-BE 경로로 옮긴 커밋입니다. 작성자는 원저자로 유지했습니다.SearchRequestPolicy의 import 한 줄뿐이었습니다. #33에서 추가된List는 쓰이지 않아서Set만 남겼습니다..gitignore,api/)은 제외했습니다.V7__update_saving_period_options.sql은 비어 있던 V7 번호에 그대로 들어갑니다. 운영 DB가 아직 없어서 V8 다음에 V7이 적용되는 out-of-order 문제는 없습니다.0c42380: 이관하면서 생긴 의미상 충돌을 수정했습니다(아래).버그 해결 💊
원본 PR은 예금과 적금이
min_monthly_limit을 같이 쓰던 시점을 전제로 작성됐습니다. #32에서 예금 한도가min/max_deposit_amount로 분리된 뒤로는 금액 하한 검증이 다음처럼 잘못 동작했습니다.min_monthly_limit이 null이라 하한 검사가 사실상 꺼져 있었습니다.min_monthly_limit과 비교하고 있었습니다. V8 주석에 따르면 파킹도min_deposit_amount를 씁니다.수정 후에는 상품 유형에 맞는 금액끼리 비교합니다. 금액은 원본 PR에 있던
normalizedMonthlyDeposit/normalizedDepositPrincipal로 정규화합니다.min_deposit_amountmin_monthly_limitmin_deposit_amount목돈 예금도 이번 수정부터
min_deposit_amount로 검증됩니다. #32 이후로는 검증되지 않던 부분입니다.테스트 💻
EligibilityBoundaryIntegrationTest에 예치금 하한 경계 케이스 5개를 추가했습니다(단기 예금, 단기 적금, 목돈 예금). 수정 전에는 3개가 실패하는 것을 확인했습니다.ParkingProductServiceTest의 하한 테스트가minDepositAmount를 보도록 바꿨습니다../gradlew test(349개)와./gradlew integrationTest(123개)가 모두 통과합니다.비고 ✏
원본 PR 리뷰에서 남은 항목(파킹 카드 필드를 V8 컬럼과 연결, 파킹 개수 중복 카운트, 정렬 테스트 정리)은 이 PR 위에 올리는 후속 PR에서 처리합니다.