Skip to content

[13.x] Add an optional migration for oauth_clients table to the upgrade guide#1843

Merged
taylorotwell merged 3 commits intolaravel:13.xfrom
hafezdivandari:13.x-upgarde-guide-for-new-clients-table
Aug 6, 2025
Merged

[13.x] Add an optional migration for oauth_clients table to the upgrade guide#1843
taylorotwell merged 3 commits intolaravel:13.xfrom
hafezdivandari:13.x-upgarde-guide-for-new-clients-table

Conversation

@hafezdivandari
Copy link
Copy Markdown
Contributor

@taylorotwell taylorotwell merged commit d1c21a2 into laravel:13.x Aug 6, 2025
8 checks passed
@hafezdivandari hafezdivandari deleted the 13.x-upgarde-guide-for-new-clients-table branch August 6, 2025 18:29
@majchrosoft
Copy link
Copy Markdown

majchrosoft commented Sep 8, 2025

$table->nullableMorphs('owner')->after('user_id');


  Call to a member function after() on null

  at database/migrations/2025_08_20_123028_passport_manual_update.php:16
     12▕      */
     13▕     public function up(): void
     14▕     {
     15▕         Schema::table('oauth_clients', function (Blueprint $table) {
  ➜  16▕             $table->nullableMorphs('owner')->after('user_id');
     17▕ 
     18▕             $table->after('provider', function (Blueprint $table) {
     19▕                 $table->text('redirect_uris');
     20▕                 $table->text('grant_types');
     
     
    

valid use of nullableMorphs


            $table->nullableMorphs('owner', null, 'user_id');

@hafezdivandari
Copy link
Copy Markdown
Contributor Author

hafezdivandari commented Sep 15, 2025

Thanks @majchrosoft PR #1853 has been submitted to resolve this.

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.

No migration template in the UPGRADE guide

3 participants