mollie-api-php icon indicating copy to clipboard operation
mollie-api-php copied to clipboard

Workaround for testmode query param on cancel subscription call

Open ronaldtb opened this issue 4 years ago • 4 comments

Hi,

I've left a comment on #437 and this pull request contains a workaround for the Error executing API call (422: Unprocessable Entity): Non-existent query parameter "testmode" for this API call.. Field: testmode exception on the subscription->cancel() call.

In https://github.com/mollie/mollie-api-php/issues/437#issuecomment-580770762 it's mentioned that Mollie is accepting the testmode query parameter on the other endpoints, and I think adding this support also to the cancel subscription endpoint would be a better solution than merging this workaround.

ronaldtb avatar Mar 18 '21 09:03 ronaldtb

Thanks @ronaldtb !

In #437 (comment) it's mentioned that Mollie is accepting the testmode query parameter on the other endpoints, and I think adding this support also to the cancel subscription endpoint would be a better solution than merging this workaround.

I agree, I'll discuss this option with the Mollie team first

sandervanhooft avatar Mar 18 '21 12:03 sandervanhooft

Hang on @ronaldtb, we're still looking into this 😄

sandervanhooft avatar Mar 29 '21 16:03 sandervanhooft

Mollie will solve this in the API. I'll leave this PR open for reference until the issue has been solved.

sandervanhooft avatar Mar 30 '21 07:03 sandervanhooft

Thanks for the update @sandervanhooft. I'm currently using the workaround from my pull request, so there's no hurry from my side.

ronaldtb avatar Mar 30 '21 10:03 ronaldtb

seems like the issue was fixed

Naoray avatar Sep 14 '23 09:09 Naoray