-
Notifications
You must be signed in to change notification settings - Fork 12
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
[CC-1377] add new paypage #192
Conversation
- Create basic paypage v2 resource - add authorization handling for different apis. - Add basic integration test.
- add properties with primitive data types.
- Add Urls, Style and Paymentmethodsconfigs properties.
- Add resources property.
New Issues
|
- enable paypage fetching.
- assign test groups
- assign test groups
- assign test groups.
- update methodConfig - Add method to easier add method config.
- add test with label method config.
- Add workflow to run specific test groups (including unit and integration tests)
$unzer = $this->getUnzerObject(); | ||
$risk = new RiskData(); | ||
|
||
$risk->setThreatMetrixId('f544if49wo4f74ef1x') |
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.
We only support these fields in risk:
"risk": {
"registrationLevel": "string",
"registrationDate": "string",
"customerGroup": "top",
"confirmedOrders": 0,
"confirmedAmount": 0
}
customerId and threatmetrixId are gone.
ThreadMetrixId will be generated by the Paypage UI
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.
@AntonAtUnzer
I see, I reused the risk object for normal transactions here. So i can not remove them from the original class.
The merchant could just not set the properties. Or should i make sure that he is not setting them for paypage requests?
src/Resources/EmbeddedResources/Paypage/PaymentMethodConfig.php
Outdated
Show resolved
Hide resolved
src/Resources/V2/Paypage.php
Outdated
protected float $amount; | ||
protected string $currency; | ||
|
||
protected ?string $type = null; |
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.
checkoutType as well e.g. full, payment_only, no_shipping
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.
- Add checkout type to paypage.
- Remove checkout type from method config.
- Add checkout type to paypage. - Remove checkout type from method config.
- Split up riskdata object to have one version for normal transaction requests and one for paypage creation requests.
No description provided.