-
Notifications
You must be signed in to change notification settings - Fork 30
V1 테스트 채널 어뷰징 대응 관련 API 변경사항 적용 #982
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @SonnySon17, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 V1 API의 테스트 채널 어뷰징 대응과 관련된 변경사항을 적용합니다. 오래된 TLS 보안 공지를 제거하고, 결제내역 및 빌링키 조회 응답에 'sandbox' 필드를 추가하여 테스트 채널을 통한 거래 여부를 명확히 식별할 수 있도록 합니다. 또한, 결제내역 단건 조회 시 테스트 채널 결제건을 포함할지 여부를 제어하는 'include_sandbox' 파라미터를 도입하여 API 사용성을 개선합니다. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
V1 API의 테스트 채널 어뷰징 대응을 위한 변경사항을 적용하는 PR이군요. 오래된 TLS 공지를 제거하고, 결제 및 빌링키 응답에 sandbox 필드를 추가하며, 결제 조회 API에 include_sandbox 파라미터를 추가하는 내용이 잘 반영되었습니다. 전반적으로 변경 사항은 좋아 보입니다. OpenAPI 명세의 명확성을 높이기 위해 새로운 파라미터에 기본값을 명시적으로 정의하는 것을 제안하는 몇 가지 의견을 남깁니다.
| { | ||
| "name": "include_sandbox", | ||
| "in": "query", | ||
| "description": "테스트 채널을 통해 발생한 결제건을 조회 결과에 포함할지 여부 (default: false)", | ||
| "required": false, | ||
| "type": "boolean", | ||
| "x-portone-name": "테스트 결제건 포함 여부", | ||
| "x-portone-description": "테스트 채널을 통해 발생한 결제건을 조회 결과에 포함할지 여부 (기본값: `false`)\n\n(본 파라미터는 `2026년 1월 26일`부터 추가 예정입니다.)" | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
include_sandbox 파라미터의 description에 (default: false)가 언급되어 있습니다. OpenAPI 명세에서 default 필드를 사용하여 기본값을 명시적으로 정의하는 것이 좋습니다. 이렇게 하면 API 계약이 더 명확해지고 코드 생성기와 같은 도구에서 올바르게 활용할 수 있습니다.
{
"name": "include_sandbox",
"in": "query",
"description": "테스트 채널을 통해 발생한 결제건을 조회 결과에 포함할지 여부",
"required": false,
"type": "boolean",
"default": false,
"x-portone-name": "테스트 결제건 포함 여부",
"x-portone-description": "테스트 채널을 통해 발생한 결제건을 조회 결과에 포함할지 여부 (기본값: `false`)\n\n(본 파라미터는 `2026년 1월 26일`부터 추가 예정입니다.)"
}| - name: include_sandbox | ||
| in: query | ||
| description: '테스트 채널을 통해 발생한 결제건을 조회 결과에 포함할지 여부 (default: false)' | ||
| required: false | ||
| type: boolean | ||
| x-portone-name: 테스트 결제건 포함 여부 | ||
| x-portone-description: |- | ||
| 테스트 채널을 통해 발생한 결제건을 조회 결과에 포함할지 여부 (기본값: `false`) | ||
| (본 파라미터는 `2026년 1월 26일`부터 추가 예정입니다.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
include_sandbox 파라미터의 description에 (default: false)가 언급되어 있습니다. OpenAPI 명세에서 default 필드를 사용하여 기본값을 명시적으로 정의하는 것이 좋습니다. 이렇게 하면 API 계약이 더 명확해지고 코드 생성기와 같은 도구에서 올바르게 활용할 수 있습니다.
- name: include_sandbox
in: query
description: '테스트 채널을 통해 발생한 결제건을 조회 결과에 포함할지 여부'
required: false
type: boolean
default: false
x-portone-name: 테스트 결제건 포함 여부
x-portone-description: |-
테스트 채널을 통해 발생한 결제건을 조회 결과에 포함할지 여부 (기본값: `false`)
(본 파라미터는 `2026년 1월 26일`부터 추가 예정입니다.)99baca5 to
3fe2edd
Compare
3fe2edd to
e04dbbe
Compare
|



sandboxfield 관련 내용 추가include_sandboxparameter 관련 내용 추가