-
Notifications
You must be signed in to change notification settings - Fork 23
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
Cannot list draft payments for monetary account #46
Comments
Hey @robinkanters, Thanks for reporting. I have a felling this has to do with 1 or multiple payments inside the LIST where |
@OGKevin that sounds correct, that is what the error says haha! |
@robinkanters btw, how did you make the draft payment ? Via API or inside the bunq app ? Save's me some time finding out how to get |
@OGKevin via the api |
@OGKevin it's just this, nothing special return DraftPayment.create(apiContext, mapOf(
"status" to "PENDING",
"entries" to listOf(mapOf(
"amount" to Amount(paymentOrder.amount.toString(), paymentOrder.currency),
"counterparty_alias" to makePointer(paymentOrder.to, paymentOrder.name),
"description" to paymentOrder.description,
"merchant_reference" to paymentOrder.uniqueKey
)),
"number_of_required_accepts" to 1
), userId, from).value |
@robinkanters I could not reproduce with the following snippet:
🤔 Later today ill try reproducing with a manually made json with the field |
@OGKevin maybe it's choking on another payment on my MA, don't know how to figure out which though |
@OGKevin try accepting the draft payment (in the app) and then listing them for that MA. It seems to choke on the geolocation on the linked payment instance for some reason. (edit: duh, that's what the error says haha)
Edit: removed ids for privacy reasons. |
never mind, that also doesn't work |
@robinkanters I think that might be it indeed. I removed the payment ids for privacy reasons. I did write them down so I’ll have a look indeed 👍. Once I’ve found a way to reproduce, I should be able to fix it and provide a test so that it doesn’t come back later on 😄. I’ll try your accept in bunq app suggestion as well, if this doesn’t work neither then I’ll try with a manual made raw json. |
who cares 😉 wouldn't have posted them if I'd have been concerned about that |
oh wait that is actually what you're doing. no idea what the difference is then |
@robinkanters reproduced 👍. Ill plan to fix this in |
@OGKevin no doubt you noticed, but just for completeness: this also happens for listing |
@robinkanters yes your indeed correct! It was due to a silly miss type 🤦♂️ |
…ocation Cannot list payments due to geolocation. (#46 )
Steps to reproduce:
What should happen:
What happens:
Logs
The text was updated successfully, but these errors were encountered: