Skip to content

Support for Laravel Passport v13#40

Open
wojo1206 wants to merge 2 commits into
corbosman:masterfrom
wojo1206:master
Open

Support for Laravel Passport v13#40
wojo1206 wants to merge 2 commits into
corbosman:masterfrom
wojo1206:master

Conversation

@wojo1206

Copy link
Copy Markdown

Updated composer to allow for Laravel Passport v13.

@what-the-diff

what-the-diff Bot commented Jun 18, 2025

Copy link
Copy Markdown

PR Summary

  • Update to the Changelog
    • This PR introduces changes to our Changelog document. It primarily adds an entry for the new version 7.1.0, which supports Laravel Passport 13, an important add-on to our system. In addition to that, it revises the compatibility notifications for Version 2.0.0 in relation to different Passport versions for clarity purpose and keep users well-informed.
  • Composer Configuration Update
    • We've tailored our system's composer.json file to accept a new version of the Laravel Passport (version 13.0). This means our system now officially supports this version and can work seamlessly with applications using it.

@corbosman

Copy link
Copy Markdown
Owner

It's not as easy as changing the composer.json unfortunately. Some code changes are necessary. I committed some changes that seem to fix the tests, but Im no longer running a laravel passport install myself. Could you see if the master branch works? We also can't use version 7.1 as the code is not backwards compatible due to changes in laravel passport. We'd have to do a 8.0.0

@wojo1206

Copy link
Copy Markdown
Author

I use Laravel v12 with Passport v13 using this fork and this lib works as expected. What other changes are necessary? I can bump the major version.

@corbosman

Copy link
Copy Markdown
Owner

The test failed due to changes in the methods on the passport side.

@rassloff

Copy link
Copy Markdown

i am waiting for the new version. so that I can use Passport 13. and other updates ...

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

Successfully merging this pull request may close these issues.

3 participants