페이팔 v2 portone-ui-container 설명 추가, 유의사항 추가 #378
Merged
Conversation
This file contains 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
Contributor
youjmen
commented
Mar 27, 2024
youjmen
commented
- 페이팔 loadUI시 portone-ui-container 에 관한 설명이 없어 추가, 정기결제 시 테스트 채널 관련 유의사항을 추가한 PR입니다.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
eslint --fix할 때 |
|
pnpm i 하고 다시 eslint 돌려보니 잘 되네요 감사합니다 👍 |
XiNiHa
reviewed
Mar 27, 2024
| @@ -51,6 +51,8 @@ import image11 from "./_assets/paypal/screenshot9.png"; | |||
| 페이팔의 경우 고객사의 결제 화면에 페이팔 결제 버튼을 렌더링 한 후 클릭해 페이팔의 결제창을 호출하는 방식입니다. | |||
| 결제 성공 시의 동작을 `onPaymentSuccess` 콜백 함수에, 결제 실패 시의 동작을 `onPaymentFail`에 작성해야 합니다. | |||
|
|
|||
| 페이팔의 결제 버튼을 렌더링 하기 위해, 버튼을 렌더링 할 곳에 className이 `portone-ui-container`인 DOM element를 선언하셔야 합니다. | |||
There was a problem hiding this comment.
Suggested change
| 페이팔의 결제 버튼을 렌더링 하기 위해, 버튼을 렌더링 할 곳에 className이 `portone-ui-container`인 DOM element를 선언하셔야 합니다. | |
| 페이팔의 결제 버튼을 렌더링 하기 위해, 버튼을 렌더링 할 곳에 클래스명이 `portone-ui-container`인 DOM element를 선언하셔야 합니다. |
아래 예시는 HTML로 작성되어 있어서 class를 사용 중이라서, className이라고 적으면 혼동이 있을 것 같아요
| 포트원 SDK는 해당 id를 가지는 element를 찾아 버튼을 렌더링합니다. | ||
| --> | ||
| <div class="portone-ui-container"> | ||
| <!-- 3. 여기에 페이팔 결제 버튼이 생성됩니다. --> |
There was a problem hiding this comment.
Suggested change
| <!-- 3. 여기에 페이팔 결제 버튼이 생성됩니다. --> | |
| <!-- 여기에 페이팔 결제 버튼이 생성됩니다 --> |
- 2.가 없어서 3.도 그냥 날려도 되지 않을까 싶어요
| @@ -122,6 +135,8 @@ function updateLoadPaymentUIRequest() { | |||
| 페이팔의 경우 고객사의 결제 화면에 페이팔 빌링키 발급 버튼을 렌더링 한 후 클릭해 페이팔의 결제창을 호출하는 방식입니다. | |||
| 발급 성공 시의 동작을 `onIssueBillingKeySuccess` 콜백 함수에, 발급 실패 시의 동작을 `onIssueBillingKeyFail`에 작성하셔야 합니다. | |||
|
|
|||
| 페이팔의 빌링키 발급 버튼을 렌더링 하기 위해, 버튼을 렌더링 할 곳에 className이 `portone-ui-container`인 DOM element를 선언하셔야 합니다. | |||
There was a problem hiding this comment.
이거 이렇게 중복 내용 2번 적는 것보다는 ## 일반 결제(loadPaymentUI) 호출하기 앞쪽에 새로 섹션 하나 분리해서 해당 섹션에 내용 서술하는 게 좋지 않을까요? ## UI 로딩 준비하기 라던가....
There was a problem hiding this comment.
일반 결제/빌링키 발급 공통 준비 사항 이라는 이름으로 중복 제거하고 공통으로 내용 추가 했습니다~
Comment on lines
1609
to
1614
| |파라미터 |설명 |예시 | | ||
| |---------|-------------------------------------------------------------------------------|-----------------------------------------------| | ||
| |`fncls` |`fnparams-dede7cc5-15fd-4c75-a9f4-36c430ee3a99`로 항상 고정 |`fnparams-dede7cc5-15fd-4c75-a9f4-36c430ee3a99`| | ||
| |`f` |주문번호(`paymentId`) 전달 |`paymentId_230980198` | | ||
| |`s` |string (`{페이팔 Account ID}_{페이지 유형}` 형식. 페이지 유형은 아래 목록 참고)|`7WBB3CKT63FRG_checkout-page` | | ||
| |`sandbox`|페이팔 Account ID가 테스트 용인지 운영 용인지 여부 |`true` | |
There was a problem hiding this comment.
Suggested change
| |파라미터 |설명 |예시 | | |
| |---------|-------------------------------------------------------------------------------|-----------------------------------------------| | |
| |`fncls` |`fnparams-dede7cc5-15fd-4c75-a9f4-36c430ee3a99`로 항상 고정 |`fnparams-dede7cc5-15fd-4c75-a9f4-36c430ee3a99`| | |
| |`f` |주문번호(`paymentId`) 전달 |`paymentId_230980198` | | |
| |`s` |string (`{페이팔 Account ID}_{페이지 유형}` 형식. 페이지 유형은 아래 목록 참고)|`7WBB3CKT63FRG_checkout-page` | | |
| |`sandbox`|페이팔 Account ID가 테스트 용인지 운영 용인지 여부 |`true` | | |
| |파라미터 |설명 |예시 | | |
| |---------|-------------------------------------------------------------------------------|-----------------------------------------------| | |
| |`fncls` |`fnparams-dede7cc5-15fd-4c75-a9f4-36c430ee3a99`로 항상 고정 |`fnparams-dede7cc5-15fd-4c75-a9f4-36c430ee3a99`| | |
| |`f` |주문번호(`paymentId`) 전달 |`paymentId_230980198` | | |
| |`s` |string (`{페이팔 Account ID}_{페이지 유형}` 형식. 페이지 유형은 아래 목록 참고)|`7WBB3CKT63FRG_checkout-page` | | |
| |`sandbox`|페이팔 Account ID가 테스트 용인지 운영 용인지 여부 |`true` | |
표가 위 list item 안에 들어가야 맥락에 맞는 활용일 것 같아서, indent를 빼기보다는 넣는 게 나을 것 같아요
XiNiHa
approved these changes
Mar 27, 2024
XiNiHa
force-pushed
the
fix/paypal-v2-description
branch
from
4156e08 to
2ddd670
Compare
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.