joselfonseca / lighthouse-graphql-passport-auth

Add GraphQL mutations to get tokens from passport for https://lighthouse-php.com/
https://lighthouse-php-auth.com/
MIT License
228 stars 56 forks source link

Add support for Laravel Passport 12 #172

Closed notf0und closed 2 weeks ago

joselfonseca commented 4 months ago

@notf0und Thanks for the PR! It seems some tests are failing. If we can fix the tests we should be able to merge this PR and create a new Release. I just don't have the time right now to take care of this. So if you can fix them that would be Great!. Also if for a new version we have to drop support for an older version of passport or laravel then we can also do that.

Saarnaki commented 4 months ago

The issue with tests failing comes from Passport 12 dropping support for automatically loading migrations.

A possible option would be to inherit the approach done by @mwangaben in his PR for supporting Passport 12?

joselfonseca commented 2 weeks ago

Done in this PR https://github.com/joselfonseca/lighthouse-graphql-passport-auth/pull/179