search

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 Lighthouse 6 #161

Closed LVMade01 closed 1 year ago

LVMade01 commented 1 year ago

Version 6.0 of Lighthouse is out. This PR will allow the new Version and adds a test for Laravel 10.

joselfonseca commented 1 year ago

@LVMade01 Thanks for the PR! I think the workflow for laravel 10 is not working. https://github.com/joselfonseca/lighthouse-graphql-passport-auth/actions/workflows/run-tests-L10.yml If that is fixed we should be good to go.

joselfonseca commented 1 year ago

@LVMade01 Actually looks like the workflow for laravel 10 is in PR https://github.com/joselfonseca/lighthouse-graphql-passport-auth/pull/162. Can we add that to this PR and make sure it works on Laravel 10 so we can have a new major version released?. Thanks for your collaboration

LVMade01 commented 1 year ago

I updated the exceptions with the code of #162 . Anyway, now in the tests a table is not found. I will look into it tomorrow.

joselfonseca commented 1 year ago

Thanks for the work @LVMade01 Compatibility for LH 6 has been added here https://github.com/joselfonseca/lighthouse-graphql-passport-auth/pull/162