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

Fixes for rails: Allow UUID/JSONB column types, ensure anonymous_class is defined #83

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tonyhb
Copy link

@tonyhb tonyhb commented May 9, 2018

  • Check for anonymous_class definition before attempting to use it
  • Ensure UUID column types are mapped to strings when using UUID PKs.
  • Add JSONB -> %jsobn type mapping.

The only thing I'm uncertain of is JSONB type mapping. As it's JSONB this should be parsed; the resulting datatype could be any primitive matching String or Integer or Float or Hash or Array or %bool. I've added a type map for %jsonb, though I'm not sure if this is correct.

@tonyhb tonyhb changed the title Allow UUID column types Fixes for rails: Allow UUID/JSONB column types, ensure anonymous_class is defined May 11, 2018
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.

1 participant