Repository navigation
마이데이터 연동 기관 캐피탈 분류 추가 및 연결 결과 정확도 수정 - #131
Merged
Merged
Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
connectedCount가 활성 여부를 안 가리고 회원의 모든 UserAccount 행을 세고 있었다. 기관을 정리(신한 제거 등)한 뒤에도 예전에 연동한 회원은 비활성화된 옛 계좌까지 세어 "5개 기관이 연결되었습니다"처럼 실제와 다른 개수가 떴다. - 활성 계좌·카드만 세도록 리포지토리 메서드를 추가했다. - connectedCount의 기준을 "계좌 행 수"에서 "중복 제거한 기관 수"로 바꿨다. 계좌만 세면 카드사(국민카드·우리카드)가 연결 목록에서 통째로 빠지는 문제도 같이 있었다. - ConnectedAccount에 accountType을 추가했다 — "모아둔 자산" 화면이 대출 잔액을 자산으로 합산하지 않으려면 프론트가 이 값으로 걸러야 한다. - Institution.KB_CARD 표시명을 "KB국민카드"에서 "국민카드"로 맞췄다 (data.sql의 카드 상품명과 통일). Co-Authored-By: Claude <noreply@anthropic.com>
은행·카드 옆에 캐피탈(현대·롯데·KB·우리금융·하나) 분류를 새로 추가했다. 분류마다 실제로 데모 데이터가 붙는 기관(국민은행·국민카드·우리카드· 현대캐피탈)을 배열 맨 앞 대표 항목으로 둬서, 뭘 고르든 이 4곳만 연동된다는 사실과 화면 순서가 어긋나지 않게 했다.⚠️ 캐피탈 로고는 실제 브랜드 자산을 받아온 게 아니다 — 이미지를 내려받는 도구가 없어서, 기존 로고와 같은 자리에 쓸 수 있게 회사명 워드마크를 간단히 그려 넣은 자리표시자다. 실제 로고 파일이 생기면 같은 파일명으로 바꿔치면 된다. Co-Authored-By: Claude <noreply@anthropic.com>
- MydataDoneView: 계좌 목록에서 이름을 직접 중복제거하던 로직을 지우고, 백엔드가 새로 내려주는 institutions(계좌+카드 통합, 중복제거)를 그대로 쓴다. 카드사가 계좌 목록에 안 잡혀 목록에서 통째로 빠지던 문제와, "5개 연결됨"처럼 실제 개수와 다르게 뜨던 문제가 같이 해결된다. - AssetsSavedView: 대출 계좌를 자산 합계에서 뺀다. 대출 잔액은 빚이지 모아둔 돈이 아니다. Co-Authored-By: Claude <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
dday | e431fd2 | Commit Preview URL Branch Preview URL |
Sep 09 2026, 08:18 AM |
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
1 participant
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.
Summary
371829)로 변경connectedCount)이 비활성 계좌까지 세거나 카드사를 누락하던 버그 수정 — 계좌·카드를 통합해 중복 제거한institutions필드를 새로 내려줌Test plan
auth/mydata도메인 테스트 전체 통과/api/mydata/connectcurl로connectedCount: 4,institutions4곳(국민은행·현대캐피탈·국민카드·우리카드) 확인vue-tsc --noEmit,eslint통과Co-Authored-By: Claude noreply@anthropic.com