Skip to content

Support for UUID in User Table, and Laravel 9 support - #17

Open
lrljoe wants to merge 4 commits into
jamesmills:masterfrom
lrljoe:master
Open

Support for UUID in User Table, and Laravel 9 support#17
lrljoe wants to merge 4 commits into
jamesmills:masterfrom
lrljoe:master

Conversation

@lrljoe

@lrljoe lrljoe commented Jun 20, 2022

Copy link
Copy Markdown

No description provided.

@lrljoe

lrljoe commented Jan 16, 2023

Copy link
Copy Markdown
Author

@jamesmills are you still maintaining this?

@jamesmills

Copy link
Copy Markdown
Owner

@lrljoe if we update the swap the bigInteger to uuid for the user_id on the watch_table then we're going to break everything aren't we?

$table->bigInteger('user_id')->unsigned()->index();
$table->uuid('user_id')->index();

@jamesmills

Copy link
Copy Markdown
Owner

@jamesmills are you still maintaining this?

I guess so, I'm not using it in any of my own projects any more.

@lrljoe

lrljoe commented Feb 19, 2023

Copy link
Copy Markdown
Author

@lrljoe if we update the swap the bigInteger to uuid for the user_id on the watch_table then we're going to break everything aren't we?

$table->bigInteger('user_id')->unsigned()->index();
$table->uuid('user_id')->index();

Errr shouldn't do (I think)? I use uuids in my fork as well as multiple user models, I'll spin up a variant tomorrow and see how broken it is with a normal bitInteger approach

@lrljoe

lrljoe commented Oct 31, 2023

Copy link
Copy Markdown
Author

Will do a re-review of this in the next month or so, make sure that I'm not breaking anything, then update you @jamesmills

Appreciate you're not using it any more, I'll do the relevant tests on my end to make sure it doesn't break anything. If it's non-breaking change, then happy to help maintain going forward as much as you want/need.

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.

2 participants