gadelkareem / sails-dynamodb

Amazon DynamoDB adapter for Waterline / Sails.js
http://sailsjs.org
46 stars 22 forks source link

Using dynamo mapped object type property attributes #27

Open tyronedougherty opened 8 years ago

tyronedougherty commented 8 years ago

Hi, does anyone know how to use attributes that are Maps in DynamoDB with waterline?

The JSON type doesn't appear to work 😕

Many thanks to anyone who may be able to help!

tyronedougherty commented 8 years ago

Correct me if I'm wrong but after much digging, it appears that JSON types are not supported?

Line 1126 of index.js (_setColumnType):

case "array":  // not support
          schema.StringSet(name, options);
          break;

             //case "json":
             //case "string":
             //case "binary":
        case "string":
             ....

Is there any chance of supporting JSON attributes in the near future?

devinivy commented 8 years ago

Definitely would take a PR!

tyronedougherty commented 8 years ago

Thanks for clarifying @devinivy, in my quick assessment it seems like it may be a significant update and may require updating the vogels package (which is currently very out of date).

devinivy commented 8 years ago

Yeah, this module needs a makeover. Some day I'll have the time to do it, but for the time being I can just support by reviewing PRs.