laravel / passport

Laravel Passport provides OAuth2 server support to Laravel.

Home Page:https://laravel.com/docs/passport

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Some tables does not meet Laravel standards

nastoychev opened this issue · comments

In whole Laravel Ecosystem the tables start with id of type bigIncrements and have by default timestamps.
Here this convention is not followed.
The problem what I have is reduced combability with Laravel Nova beacuse the action_events table does not support id of type varchar(100).

The tables are:

  • oauth_auth_codes: no id of type bigIncrements, no timestamps
  • oauth_access_tokens: no id of type bigIncrements
  • oauth_refresh_tokens: no id of type bigIncrements, no timestamps

In my opinion, the problem applies both to this project and to a Laravel/Nova one that does not allow support for other types of Ids.

In my opinion, the problem applies both to this project and to a Laravel/Nova one that does not allow support for other types of Ids.

Where exactly in Laravel Nova does it say you can only use auto increment IDs.

The only limitations is via Action Events but I don't believe you have any requirements to edit or run actions on those tables.

Welcoming PRs, thanks.