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

Generated migrations do not work if you are using UUIDs as primary key type #38

Open
johnmcdowall opened this issue May 20, 2023 · 1 comment

Comments

@johnmcdowall
Copy link

Hello, and thanks for this awesome gem!

Just wanted to let you know that the default generated migrations will silently fail if the app is using UUIDs as the primary key type. No failure occurs, just that the association values will be 0 since the default ID type is bigint.

Might be worth adding a note in the README that if you're app is using UUIDs then you'll need to edit the generated migrations. Or you could do something fancier.

Happy to submit PRs for either or neither!

Thanks 🙇

@joshmn
Copy link
Owner

joshmn commented May 21, 2023

Helloooo! No no, thank you for using it! :')

That sounds.... correct, or expected. By all means, submit submit submit! Happy to approve; I have no experience with UUIDs. :)

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

2 participants