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

Lighthouse v6 support #162

Closed wimski closed 1 year ago

wimski commented 1 year ago

This will require a major version release.

Related

LVMade01 commented 1 year ago

My branch is now working. Anyway your solution with the Exceptions look much better than mine! We should go forward with your version, but maybe you can get some additional fixes from my code.

joselfonseca commented 1 year ago

@wimski Thanks for the PR! I've removed the need for workflows in laravel 8, 7 and 6 since the only supported versions at the moment are 9 and 10. https://laravelversions.com/en We do need to fix the tests for the rest of the workflows and then we can consider this PR for the next major release.

wimski commented 1 year ago

@joselfonseca I have refactored the tests and workflow including a Docker setup to check all workflows locally. Everything passes now, but there is this weird lingering P8.2 - L9.* - prefer-stable workflow I can't get rid of.