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

this package is not supported for laravel 7.can u please provide one for laravel 7 #40

Open
aleeshanoushad opened this issue Apr 28, 2020 · 10 comments

Comments

@aleeshanoushad
Copy link

image
this is what happening when i try to install this package

@a-zog
Copy link

a-zog commented May 20, 2020

Any updates regarding this issue please?

@aleeshanoushad
Copy link
Author

Any updates regarding this issue please?

no yet

@SaeedNikmehr
Copy link

i have the same problem . for laravel 7 i had to use jens segers/mongo db version 4.* . but it seems this package can only work with v3.6 or less than that . so any one has any idea how fix this problem to use passport in laravel with mongo?

@alasdairmackenzie
Copy link
Member

Can you make a PR to resolve this?

@SaeedNikmehr
Copy link

SaeedNikmehr commented May 22, 2020

Can you make a PR to resolve this?

it seems this error happens because passport still use Illuminate\Database\Eloquent\Model instead of Jenssegers\Mongodb\Eloquent\Model
so after install jenssegers/laravel you can replace them in vendor files .
i found these :
/vendor/laravel/passport/src/AuthCode.php
/vendor/laravel/passport/src/Client.php
/vendor/laravel/passport/src/PersonalAccessClient.php
/vendor/laravel/passport/src/RefreshToken.php
/vendor/laravel/passport/src/Token.php
also check this
https://github.com/sadnub/laravel-mongodb-passport-fix

@alasdairmackenzie
Copy link
Member

Thanks for that - that looks good. I'll have a look tomorrow at those changes. If you can make a PR with those changes then that'll help me out.

@mykeysr
Copy link

mykeysr commented Jun 16, 2020

Any update please?

@hypnodev
Copy link

I've released PR #46 that with some upgrade of some dependencies make it's usable also in 7

@a-zog
Copy link

a-zog commented Feb 14, 2021

Seems that the issue is not resolved yet.

I'd like to thank the #DesignMyNight team for the great work it has done to make our lives easier.

At the same time, Laravel releases are out at a very faster pace and this package is holding our technical capabilities back.

Would you please tell us more about the challenges that are preventing you from making this 📦 available to L7 or even L8? How can this community help you go faster?

Thank you in advance

@tatyanakoba
Copy link

Hi guys, do you plan to upgrade package for new laravel version?

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

7 participants