laratrust icon indicating copy to clipboard operation
laratrust copied to clipboard

Add support for UUID on user ID foreign key constraint

Open franko172000 opened this issue 2 years ago • 2 comments

This PR adds support for users table that uses UUID as primary keys instead of the default integer type

franko172000 avatar Jul 04 '23 15:07 franko172000

If people want to use uuid or any other tyoe as primary key they have to implement it. by editing default migration which we can get from this lib. I don't think that this is usefull.

Check any other libs and they all use integer as the primary key. but developer always can change it to uuid to whetever they want

websitevirtuoso avatar Aug 29 '23 02:08 websitevirtuoso

If people want to use uuid or any other tyoe as primary key they have to implement it. by editing default migration which we can get from this lib. I don't think that this is usefull.

Check any other libs and they all use integer as the primary key. but developer always can change it to uuid to whetever they want

While I understand that developers can always change it to UUID in their app, this PR was created so that the package can support it out-of-the-box. However, if you still insist it's useless, I can always close it.

franko172000 avatar Aug 29 '23 10:08 franko172000

I agree with @websitevirtuoso closing this for now

santigarcor avatar Mar 01 '24 23:03 santigarcor