Skip to content

Conversation

jkurra-hw
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Nov 17, 2020

Coverage Status

Coverage increased (+0.1%) to 99.767% when pulling 475545e on feature-v3/DTRUNETWO-298-V3JAVASDK-TransferRefunds into f03ad20 on DEV-v3.

Copy link
Contributor

@Rrathinasabapath Rrathinasabapath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pls add the requested comment for a method and push the code.

@@ -2233,4 +2302,8 @@ private HyperwalletVenmoAccount copy(HyperwalletVenmoAccount venmoAccount) {
return venmoAccount;
}

private HyperwalletTransferRefund copy(HyperwalletTransferRefund transferRefund) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add comments for this method.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving as the comments for other similar methods will be added in the next release.

Copy link
Contributor

@Rrathinasabapath Rrathinasabapath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

…rRefunds

# Conflicts:
#	src/main/java/com/hyperwallet/clientsdk/Hyperwallet.java
#	src/test/java/com/hyperwallet/clientsdk/HyperwalletIT.java
#	src/test/java/com/hyperwallet/clientsdk/HyperwalletTest.java
@akalichety-hw akalichety-hw merged commit c509957 into DEV-v3 Nov 20, 2020
@arao6 arao6 deleted the feature-v3/DTRUNETWO-298-V3JAVASDK-TransferRefunds branch February 1, 2021 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants