search

bunq / sdk_php

PHP SDK for bunq API
MIT License
84 stars 54 forks source link

In addCallbackUrl voor Tinker geeft error Bunq PHP #161

Closed kaspar-nl closed 5 years ago

kaspar-nl commented 5 years ago

ISSUE #1 Als je de Tinker add-callback-url.php test script uitvoert, dan geeft hij een error (zie hieronder). Het lijkt erop dat er een veld niet goed staat. Waarschijnlijk ivm het ontbreken van een '$userPersonId' in 'UserPerson::update' (file: Bunqlib.php).

(allUpdatedNotificationFilterPHP Fatal error: Uncaught TypeError: Argument 28 passed to bunq\Model\Generated\Endpoint\UserPerson::update() must be an instance of bunq\Model\Generated\Object\Amount or null, array given, called in /bunq-live/src/BunqLib.php on line 476 and defined in /bunq-live/vendor/bunq/sdk_php/src/Model/Generated/Endpoint/UserPerson.php:698)

Hoe kunnen we dit oplossen?

ISSUE #2 Als je zelf je eigen id als integer toevoegt (zie ISSUE 1) , en je vult als callback URL http://www.ns.nl dan krijg je deze error: Error message: Invalid URL "http://www.ns.nl" for field "notification_target" for "notification_delivery_method" "URL".

OGKevin commented 5 years ago

I don't understand why you opened this issue in the SDK repo instead of the tinker repo :P.

Your first issue will most likely be solved by running composer update. I have a feeling that the version in composer lock of tinker is containing an outdated SDK version. If this is the case, then please open an issue in the tinker repo instead of SDK, as its not a SDK bug.

You're second issue would be solved by using HTTPS instead of HTTP.

kaspar-nl commented 5 years ago

Thanks for your feedback! It was my first post on GitHub, so I am more than happy to hear your suggestions!