Skip to content

Conversation

@puzzledpolymath
Copy link
Contributor

@puzzledpolymath puzzledpolymath commented Jul 14, 2025

This is a chess move, thinking ahead when needing to support ULID's, in preparation for https://github.com/cycle/entity-behavior-identifier see 🎟️ cycle/entity-behavior-uuid#13

Needing this and cycle/database#233 merged in order finish implementing ULID's within https://github.com/cycle/entity-behavior-identifier

🔍 What was changed

Not allot, just added methods for determining whether a type matches ulid and creating a ulid type column.

📝 Checklist

Copy link
Member

@roxblnfk roxblnfk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like we need to bump up the cycle/database dependency to ^2.14.

@puzzledpolymath
Copy link
Contributor Author

@roxblnfk Will cycle/database#230 or a fix for this problem make it into the 2.14 release? It's currently a pain point and I would prefer not having to override the class.

@puzzledpolymath puzzledpolymath merged commit 799fa1b into 1.x Jul 14, 2025
28 checks passed
@puzzledpolymath puzzledpolymath deleted the registry-modifier-ulid branch July 14, 2025 09:39
@puzzledpolymath
Copy link
Contributor Author

@roxblnfk Is it possible to add this into a new release?

@roxblnfk
Copy link
Member

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.

3 participants