-
Notifications
You must be signed in to change notification settings - Fork 63
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
Update payment not supported? #128
Comments
That is correct: it has not been implemented in this library yet. There are plans to fill up the gaps in this library in the near future. |
Okay. I thought I'd give it a go but I ran into some problems. Got lots of these when I tried running the tests:
And I don't see any documentation about contributing so not sure what to do here. Also you might want to switch from yarn back to npm because I think Facebook kind of abandoned yarn, there's just 1 developer on it now that made yarn v2. |
@Christilut did you made any progress on this in the meantime? |
I just do a REST call manually for the UPDATE payment: export async function sendMollieRest(method: 'PATCH', url: string, data: object) {
if (env.NODE_ENV !== env.Environments.Test) {
await Axios({
method,
url,
headers: {
Authorization: `Bearer ${env.MOLLIE_API_KEY}`
},
data
})
}
} |
We have a PR ready. We plan to make a new beta release which includes this endpoint on Monday. |
@Christilut thanks! @Pimm Thats great, I can wait for Monday :) |
@Christilut @lmolema We just released Please note that this is a beta. Let us know if you run into any issues! |
@vernondegoede Great! I am testing it and so far it works as expected and no issues! |
It says the Payment update() method is not supported in the v2 API, but the v2 API does show this method: https://docs.mollie.com/reference/v2/payments-api/update-payment
Is it just not implemented yet in the nodejs client?
The text was updated successfully, but these errors were encountered: