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
Feature/dtpayetwo 759 hw parity paypal account java dev #194
Feature/dtpayetwo 759 hw parity paypal account java dev #194
Conversation
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.
LGTM
… feature/DTPAYETWO-759-HWParity_PaypalAccountJavaDev # Conflicts: # CHANGELOG.md # src/test/java/com/hyperwallet/clientsdk/HyperwalletIT.java
@@ -3377,6 +3377,93 @@ public void testGetPayPalAccountDefaultTransfer() throws Exception { | |||
String functionality = "getPayPalAccountDefaultTransfer"; | |||
initMockServer(functionality); | |||
|
|||
@Test |
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.
The testGetPayPalAccountDefaultTransfer()
method is missing code.
@@ -3377,6 +3377,93 @@ public void testGetPayPalAccountDefaultTransfer() throws Exception { | |||
String functionality = "getPayPalAccountDefaultTransfer"; | |||
initMockServer(functionality); | |||
|
|||
@Test | |||
public void createPayPalAccountWithAccountId() throws Exception { |
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.
Minor The test name should be testCreatePayPalAccountWithAccountId()
JIRA Ticket: DTPAYETWO-759
JIRA Tickets:
Overview
The Design Documentation can be found here: HW Parity with Masspay (REST API)
Currently when creating External Accounts (EA) such as PayPal & Venmo through the REST API method, account numbers were specified to be the following:
This has now been changed to support the following:
With this change, there is a new option to use the "accountId" key-value pair in the HTTP POST Request Body. When GETting an EA, the response body will show this "accountId" value as well. Users are still able to use the "email" key-value if they wish to as well.
Added new wallet attribute USE_EMAIL_TAG_FOR_PAYPAL_EA_REST_API.
Hence we have added a generic error 'email/accountId is required' to accomodate both based on wallet attribute USE_EMAIL_TAG_FOR_PAYPAL_EA_REST_API. Moving forward all merchants will start using accountId.