breeze icon indicating copy to clipboard operation
breeze copied to clipboard

[2.x] Support Laravel Passport

Open hafezdivandari opened this issue 1 year ago • 4 comments

  • [x] Requires laravel/passport#1771
  • [x] Requires laravel/framework#55621

Todo

  • Add device authorization views.
  • Add tests stubs.

Features

  • First-party integration with Laravel Passport.
  • New OAuth option during installation: breeze:install --oauth
  • Supported stacks:
    • ✅ Default Blade
    • ✅ Inertia + Vue
    • ✅ Inertia + Vue + Typescript
    • ✅ Inertia + React
    • ✅ Inertia + React + Typescript
    • ❌ Livewire + Volt Class API
    • ❌ Livewire + Volt Functional API

Authorize View

laravel-breeze

hafezdivandari avatar Aug 02 '24 07:08 hafezdivandari

Thanks for submitting a PR!

Note that draft PR's are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

github-actions[bot] avatar Aug 02 '24 07:08 github-actions[bot]

This is a great addition to Laravel Breeze, bringing first-party support for Passport and simplifying OAuth setup. Looking forward to seeing the finalized version.

akr4m avatar Oct 23 '24 17:10 akr4m

This is a great additional! I'd just want to point out that, at least on the React Typescript implementation that I tested, a real form submission should be performed instead of using the useForm post method. We want to have a full page reload in this case, since we want to perform a real redirect to the redirect_uri provided.

akyrey avatar Jan 27 '25 15:01 akyrey

@akyrey Thank you for testing and reviewing! Please feel free to suggest any changes needed on this PR. However, AFAIK Breeze and Jetstream are set to be retired soon, with separate starter kits planned for each setup. You can refer to laravel/passport#1778 (comment) for more details.

hafezdivandari avatar Feb 01 '25 20:02 hafezdivandari