Support for UUID
opened this issue · comments
Doesn't seem to work with UUID strings.
Can you be more specific?
We have projects that we use UUID string instead of the incrementing int ID's. But your package doesn't support string ID's.
This: $table->morphs('authenticatable');
should be $table->uuidMorphs('authenticatable');
This will support for for both int and string id's
I see. I'm open to pull requests if you have time.
Honestly the easiest solution I think is to publish the migrations:
php artisan vendor:publish --provider="Rappasoft\LaravelAuthenticationLog\LaravelAuthenticationLogServiceProvider" --tag="authentication-log-migrations"
And update the morphs
to uuidMorphs
before you migrate. Would that fix your problem?
Honestly the easiest solution I think is to publish the migrations:
php artisan vendor:publish --provider="Rappasoft\LaravelAuthenticationLog\LaravelAuthenticationLogServiceProvider" --tag="authentication-log-migrations"
And update the
morphs
touuidMorphs
before you migrate. Would that fix your problem?
thank you. yes that would. I ended up forking your repo and added more functionalities to it. thanks for sharing your code!
Sounds good. I hope that solves it, I'm going to close the PR since it wouldn't work for everyone.