-
Notifications
You must be signed in to change notification settings - Fork 67
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
Missing endpoint payments of payment-links #381
Comments
Maybe possible to squeeze in https://github.com/mollie/mollie-api-node/milestone/6, too @Pimm? |
The main extension currently for 4.2.0 is the addition of the terminals API. Please also be advised that there is currently no fixed schedule for 4.2.0 @mattickx: We strive to keep the client as up to date as possible. The added overhead of managing missing or disabled endpoints in the code often is on par or even exceeds the efforts needed to just implement them for real. This is why we generally try to avoid this. |
Any updates on this? Im playing around with payment links. I expected the web hook call after a link payment to include the status of the payment. But it looks like I only got an id; Im used to stripe links where I could create a single link (that would collect the customer email, and the email could be prefilled with a query parameter like ?prefilled_email=...) |
We are currently still working on getting this done, either for the upcoming 4.2.0 release or the 4.3.0 release the latest.
Yes. As we send webhook payloads unsigned and unencrypted, this is for security reasons. I think there is nothing wrong with creating as many payment links as you need :) |
Add missing endpoint
https://api.mollie.com/v2/payment-links/{id}/payments
to the mollie node client.Docs:
https://docs.mollie.com/reference/get-payment-link-payments
Related discord topics:
https://discord.com/channels/1037712581407817839/1288507180122640456
https://discord.com/channels/1037712581407817839/1288137318691242058
https://discord.com/channels/1037712581407817839/1166419761060646942
https://discord.com/channels/1037712581407817839/1057291926786801735
As requested by Flo on discord, adding it here as an issue to keep track of it.
Maybe worth it to document which endpoints are missing.
Or at least implement it with a throw
throw new Error('Not yet implemented')
The text was updated successfully, but these errors were encountered: