Skip to content

Add missing OAuthenticatable interface step to upgrade guide#1887

Merged
taylorotwell merged 2 commits intolaravel:13.xfrom
FeBe95:patch-1
Feb 18, 2026
Merged

Add missing OAuthenticatable interface step to upgrade guide#1887
taylorotwell merged 2 commits intolaravel:13.xfrom
FeBe95:patch-1

Conversation

@FeBe95
Copy link
Contributor

@FeBe95 FeBe95 commented Feb 18, 2026

The upgrade guide was missing a step instructing users to add the Laravel\Passport\Contracts\OAuthenticatable interface to their App\Models\User model. This interface was introduced in Passport 13. Prior versions only required the HasApiTokens trait. Without this, upgrading users will have no indication they need to add it.

Changes

Added a new entry to UPGRADE.md under the "Upgrading To 13.0 From 12.x" section documenting the required User model change, consistent with what the installation docs already state.

FeBe95 and others added 2 commits February 18, 2026 19:00
Removed duplicate section on User Model Interface and retained relevant information.
@taylorotwell taylorotwell merged commit e9cc1d5 into laravel:13.x Feb 18, 2026
12 checks passed
@hafezdivandari
Copy link
Contributor

This is optional, that's why it is not mentioned on the upgrade guide but in the docs for new installations. Upgrade guide usually contains breaking changes only.

@FeBe95 FeBe95 deleted the patch-1 branch February 18, 2026 22:27
@FeBe95
Copy link
Contributor Author

FeBe95 commented Feb 18, 2026

@taylorotwell Thanks for merging!

@hafezdivandari We were already using the HasApiTokens trait before. The trait now includes the @phpstan-require-implements annotation to make sure that the trait is only used with OAuthenticatable classes. PHPStan started reporting errors because of this change, but I couldn’t find any mention of it in the upgrade guide. That’s why I thought it would be helpful to include it there.

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

Comments