Implement unwrap for lower-case type strings

[arichiardi]

Avoid failing with a "No data type (B, BS, N, NS, S, SS)" error when
type strings are not upper case.

[coveralls]

Coverage increased (+0.02%) to 96.532% when pulling 72033b0 on arichiardi:lowercase-support into 5bb4c7a on kayomarz:master.

[kayomarz]

@arichiardi Thanks for the pull-request.

If DynamoDB SDK doesn't support lower-case type strings, I'm not sure if its a good idea for this library to convert case. This library is just a light wrapper.

[arichiardi]

Well, I have had DynamoDB extract data in lowercase while using Data Pipeline that is why I needed it.

In any case up to you 😀

[kayomarz]

Ok. I understand. I will merge your pull request, maybe with a global flag to allow lowercase data types. Maybe something like: var ddt = require('dynamodb-data-types'); ddt.allowLowercaseDataType(); Let me know if you have any other suggestion. Thanks! On 27-Mar-2017 10:42 am, "Andrea Richiardi" <notifications@github.com> wrote: Well, I have had DynamoDB extract data in lowercase while using Data Pipeline that is why I needed it. In any case up to you 😀 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#10 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAIkCXoXr-8iMM3exO76FgKqJfjlDtYDks5rp8rLgaJpZM4MnKPQ> .

[prothschell-lowtcenter]

In my experience of importing/exporting text files into/from DynamoDB using Data Pipeline, only the first letter of the data type is set to lower case.

Additionally, it places a char(3) rather than a colon between root level field names & their data and then a char(2) between fields.

YMMV