fix: 중복 Service Id "PayInfo" 통합 (공공기관 항목으로 일원화)#139
Merged
Conversation
같은 서비스(금융결제원 계좌정보통합관리서비스, payinfo.or.kr)가 공공기관/일반 두 섹션에 중복 등록되어 Service Id "PayInfo"가 충돌했습니다. - 공공기관 항목(Category=Government)을 정본으로 유지 → 다른 금융결제원 서비스(KFTCVAN, CMSEDI 등)와 카테고리 일관성 확보 - 일반 섹션 중복 항목이 갖고 있던 MobiSign 패키지를 정본에 병합 → nProtect·MobiSign 설치 URL 모두 생존(HTTP 200, .exe) 확인 - 일반 섹션 중복 항목 제거 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR resolves a duplicate Service Id="PayInfo" entry in docs/Catalog.xml by consolidating the “계좌정보통합관리서비스 (payinfo.or.kr)” definition into the Government (공공기관) section and removing the redundant “Other” section entry, while preserving plugin/package information.
Changes:
- Added the
MobiSignpackage to the GovernmentPayInfoservice entry. - Removed the duplicate
PayInfoservice entry from the Other section, eliminating theService Idcollision.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
문제
같은 서비스(금융결제원 계좌정보통합관리서비스,
payinfo.or.kr)가 카탈로그의 공공기관과 일반 서비스 두 섹션에 각각 등록되어Service Id="PayInfo"가 중복됩니다.Catalog.xsd는 Service Id의 부모 기준 유일성을 요구합니다..../...해결
Government라 카테고리 일관성 확보.MobiSign패키지를 A에 병합 — 정보 손실 방지. nProtect·MobiSign 설치 URL 모두 생존(HTTP 200,.exe) 확인.검증
Service Id없음 (sort | uniq -d결과 없음) ✔관련: PR #135 리뷰 중 발견된 기존 중복 이슈의 후속 처리.
🤖 Generated with Claude Code