-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UUID v7 support / ULID support? #119
Comments
Happy to review a PR, but also considering that UUID/ULID support is coming to Laravel, need to consider what versions of Laravel we support into the future as well. |
Ahh ok, do you have a link to that ticket? |
This is the PR that introduced the functionality, so it's available in Laravel 9.x. Laravel 10.x will use UUIDv7 by default. |
What about ulid? Uuid are great, but ulids are more compact ant url friendly. Can we create ulid specific mr? |
PR'd #124 Looks like laravel reverted v7 becoming default due to BC breaks. |
Laravel 9.x has ULID support as well. https://laravel.com/docs/9.x/migrations#column-method-ulid Methods are:
|
@michaeldyrynda will you accept a ULID PR as well? |
PR'd #127 |
@michaeldyrynda I know this is a little old but Laravel did revert the PR and decided not to use UUIDv7 by default. It would be nice if we could set a config to choose the default UUID instead of having to create a trait to override the default on every model. |
Thoughts on adding v7/ulid support?
https://uuid.ramsey.dev/en/stable/rfc4122/version7.html?highlight=encoding#
Happy to PR :)
The text was updated successfully, but these errors were encountered: