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 based PK are not supported #60

Open
etcook opened this issue Sep 29, 2018 · 0 comments
Open

UUID based PK are not supported #60

etcook opened this issue Sep 29, 2018 · 0 comments

Comments

@etcook
Copy link

etcook commented Sep 29, 2018

One issue for us is the assumption of serial IDs. We have already implemented UUIDs as a primary key for all of our models, but many of the underlying methods normalize against the integer assumption. This makes things like account_summary, break.

Before I work on a PR, I'd like to discuss how you'd like to handle these. Would you like to conditionally test for UUIDs and then continue to normalize for integers or would you like to just sanitize and allow both?

I'd also like to explore replicating things like polymorphic accessors, to make the objects behave more "Active Record-like."

The foundational structure is working well. Thanks for your work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants