harrygulliford / laravel-firebird

Firebird database driver for the Laravel Framework
65 stars 30 forks source link

Failed migrations on Laravel 6.x #13

Closed qlixes closed 4 years ago

qlixes commented 4 years ago

first, I'm using Laravel 6.0, then tried to install laravel-firebird. I've found some error :

  1. failed to migrations
        Schema::create('roles', function (Blueprint $table) {
            $table->useIdentity();
            $table->bigIncrements('id');
            $table->string('role_name')->unique();
            $table->boolean('all_branch')->default(false)->comment('access all branch');
            $table->boolean('is_active')->default(true);
            $table->timestamps();
        });

    because on Firebird Grammer, while had default value, it's doesn't need NOT NULL again.

Thank you a lot.

harrygulliford commented 4 years ago

@qlixes Migrations aren't something that have been properly tested and I should probably add a warning in the readme. If you do require it, I'm open to PRs.