Skip to content
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 support? #25

Open
czechbox opened this issue Jan 26, 2020 · 0 comments
Open

UUID support? #25

czechbox opened this issue Jan 26, 2020 · 0 comments

Comments

@czechbox
Copy link

Was about to implement this, and reviewing the migrations, noticed that columns like reputations.subject_id, reputations.payee_id and user_badges.user_id make assumption about column types (e.g. these are integers or unsignedIntegers).

Was going to just edit the migration, but thought there might be some value in making this more introspective, as I'm sure I won't be the only person using UUIDs for some of these.

The first hurdle seems to be it's either/or not both so some structural change may be needed to extend this to mixed ID types. Perhaps a pivot table for this relationship might make more sense.

Next hurdle is these are in effect, foreign keys. Should we care about that and take advantage of things like cascading deletes?

Before I jump into all that, I thought it might be best to find out what your thoughts might be on these things. Happy to collaborate and submit PRs, but if it's not a direction you're interested in, let me know so I don't waste your time.

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

No branches or pull requests

1 participant