-
Notifications
You must be signed in to change notification settings - Fork 660
Fix compatibility with laravel/passport #332
Conversation
Can we have this merged please? Unable to update this package in L7. |
I'm waiting for it too. |
@freekmurze @mpociot Please merge this? |
Same here. Thank you ! |
Same here please ❤️ |
Need help with this too, there is also error on Laravel 6.18.1 and "laravel/passport": "^8.4". |
waiting for this to be merged |
I just upgraded to Laravel 7 and was having the same issue however I updated this package to be ^1.4 and also installed nyholm/psr7. After installing nyholm/psr7, Passport is now working without having to upgrade the symfony/psr-http-message-bridge package. Granted I only use Passport in a very limited scope so this may not help for all use cases of Passport. It would be nice for this issue to be resolved so the Symfony package can be updated but just wanted to share my experience in case it helps anyone else. Edit: I have laravel/passport at version 8.4 |
Any progress on this pull request? Just wondering if there is a problem that is preventing a merge. More asking so that we can possibly help. |
@freekmurze @mpociot could you have a look at this PR? it's preventing a lot of us from upgrading to Laravel 7 |
For anyone who are having this issue, you could temporarily use this fork:
|
Thats how I solved! Until this is merged or the issue is fixed. |
24 days already! what are you waiting for? |
Addresses #331