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

Typed property Spatie\WebhookClient\ProcessWebhookJob::$webhookCall must be an instance of Spatie\WebhookClient\Models\WebhookCall #57

Closed
Jabirr opened this issue Apr 2, 2020 · 5 comments

Comments

@Jabirr
Copy link

Jabirr commented Apr 2, 2020

So i keep getting this error when ever im trying make a source.charge

Typed property Spatie\WebhookClient\ProcessWebhookJob::$webhookCall must be an instance of Spatie\WebhookClient\Models\WebhookCall
I get this error when at the stripe dashboard.

I am using the 2.2.1 version since im still on laravel 5.8. But i have no idea what im doing wrong?

In my web.php i added the stripe-webhook
My config is as follow:
`<?php

return [

'signing_secret' => env('STRIPE_WEBHOOK_SK'),

'jobs' => [
    'source_chargeable' => \App\Jobs\StripeWebhooks\HandleChargeableSource::class,
    'charge_succeeded' => \App\Jobs\StripeWebhooks\HandleSucceededCharge::class,
],

'model' => \Spatie\StripeWebhooks\ProcessStripeWebhookJob::class,

];
`

But it doesnt even hit on of these jobs, what am i missing?

@Jabirr
Copy link
Author

Jabirr commented Apr 2, 2020

It seems it breaks in WebhookProcessor.php line 63 where dispatch($job) gets fired, if i remove that and add dispatch(new HandleChargeableSource($webhookCall)) then it works...

@N3rdP1um23
Copy link

Hey! I have the same issue also. I'm using the Stripe CLI and tried your correction but that didn't help also....

@Jabirr
Copy link
Author

Jabirr commented Apr 14, 2020

It worked for me to downgrade to a earlier version. In the newer version they use public WebhookCall $webhookCall. Which is breaking for me. If i remove the WebhookCall and make it public $webhookCall; then it works fine

@freekmurze
Copy link
Member

Feel free to PR a fix for this.

@spatie-bot
Copy link

Dear contributor,

because this issue seems to be inactive for quite some time now, I've automatically closed it. If you feel this issue deserves some attention from my human colleagues feel free to reopen it.

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

4 participants