Skip to content
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

In addCallbackUrl voor Tinker geeft error Bunq PHP #161

Closed
kaspar-nl opened this issue Oct 9, 2018 · 2 comments
Closed

In addCallbackUrl voor Tinker geeft error Bunq PHP #161

kaspar-nl opened this issue Oct 9, 2018 · 2 comments

Comments

@kaspar-nl
Copy link

kaspar-nl commented Oct 9, 2018

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
Copy link
Contributor

OGKevin commented Oct 9, 2018

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.

@OGKevin OGKevin closed this as completed Oct 9, 2018
@kaspar-nl
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants