Skip to content
This repository has been archived by the owner on Oct 9, 2024. It is now read-only.

Using Refund with a processDate always fails #21

Open
abelbraaksma opened this issue Oct 24, 2019 · 1 comment
Open

Using Refund with a processDate always fails #21

abelbraaksma opened this issue Oct 24, 2019 · 1 comment

Comments

@abelbraaksma
Copy link

abelbraaksma commented Oct 24, 2019

For example:

// ... other code
var processDate = DateTime.Today.AddDays(1);
// always throws
var refundResponse = PAYNLSDK.Transaction.Refund(transactionId, description, amountInCents, processDate, refundNotificationUrl);  

The problem appears to be with the date being formatted as YYYY-MM-DD, but the pay.nl server expects the Dutch format, DD-MM-YYYY.

sebastianberm added a commit to sebastianberm/csharp-sdk that referenced this issue Oct 25, 2019
sebastianberm added a commit to sebastianberm/csharp-sdk that referenced this issue Oct 25, 2019
@abelbraaksma
Copy link
Author

@sebastianberm, I understand that this has been fixed now and that it has been tested? Shall I (or you) close this issue?

sebastianberm added a commit to sebastianberm/csharp-sdk that referenced this issue Dec 17, 2019
sebastianberm added a commit to sebastianberm/csharp-sdk that referenced this issue Dec 17, 2019
sebastianberm added a commit that referenced this issue Jan 6, 2020
* Fixing #21

* Bugfix release

Fixed issue #21

* Fix for #23

Fix for the PaymentOptionSubId, which can now be both an int or a string depending on usage.

Furthermore, upgraded the dependancy to the latest version.
dampee referenced this issue in roodfluweel/csharp-sdk Mar 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant