-
Notifications
You must be signed in to change notification settings - Fork 26
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
feat: add samsung pay payment method support for cybersource #1650
base: main
Are you sure you want to change the base?
feat: add samsung pay payment method support for cybersource #1650
Conversation
Review changes with SemanticDiff. |
->getArrayFromDict("countries", []) | ||
->Array.map(item => { | ||
let dict = item->getDictFromJsonObject | ||
let a: SelectBox.dropdownOption = { |
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.
pls update the variable name
@@ -1404,6 +1407,7 @@ let itemToPMAuthMapper = dict => { | |||
let constructConnectorRequestBody = (wasmRequest: wasmRequest, payload: JSON.t) => { | |||
open LogicUtils | |||
let dict = payload->getDictFromJsonObject | |||
Js.log2(dict, "DICT") |
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.
please remove this log
let samsungPayFields = React.useMemo(() => { | ||
try { | ||
if connector->isNonEmptyString { | ||
let dict = |
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.
update variable name
@@ -111,7 +112,8 @@ let make = ( | |||
</h2> | |||
</div> | |||
<div className="self-center"> | |||
<Button text="Proceed" buttonType={Primary} onClick={_ => onSubmit()->ignore} /> | |||
// <Button text="Proceed" buttonType={Primary} onClick={_ => onSubmit()->ignore} /> |
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.
remove comment
@@ -152,6 +152,8 @@ let getPaymentMethodTypeFromString = paymentMethodType => { | |||
| "apple_pay" => ApplePay | |||
| "paypal" => PayPal | |||
| "open_banking_pis" => OpenBankingPIS | |||
| "samsung_pay" => SamsungPay | |||
|
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.
empty line addition not required here
| Select => selectInput(~field={samsungPayField}, ~formName) | ||
| MultiSelect => multiSelectInput(~field={samsungPayField}, ~formName) | ||
| Radio => radioInput(~field={samsungPayField}, ~formName, ~fill, ()) | ||
|
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.
pls remove empty line addition
setScreenState(_ => Loading) | ||
let paymentMethoConfigUrl = getURL(~entityName=PAYMENT_METHOD_CONFIG, ~methodType=Get) | ||
let res = await fetchDetails( | ||
`${paymentMethoConfigUrl}?connector=${connector}&paymentMethodType=apple_pay`, |
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.
${paymentMethoConfigUrl}?connector=${connector}&paymentMethodType=apple_pay
,
Will there be paymentMethodType = applePay?
Type of Change
Description
Motivation and Context
How did you test it?
Under cybersource wallet you should find samsung pay as one of the payment method
Where to test it?
Checklist
npm run re:build