Support for UUID

[simtabi]

Doesn't seem to work with UUID strings.

[rappasoft]

Can you be more specific?

[simtabi]

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

[rappasoft]

I see. I'm open to pull requests if you have time.

[rappasoft]

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?

[simtabi]

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?

thank you. yes that would. I ended up forking your repo and added more functionalities to it. thanks for sharing your code!

[rappasoft]

Sounds good. I hope that solves it, I'm going to close the PR since it wouldn't work for everyone.

[BRGustavoRibeiro]

Hi, guys! I'm having a problem with this one. I've published the migrations and changed it to support UUID, but it is throwing me this error:

Any ideas? Thx