search

awspilot / dynamodb-oop

Speak fluent DynamoDB, write code with fashion, I Promise() 😃
https://awspilot.dev
MIT License
119 stars 28 forks source link

adding ProjectionExpression #23

Closed snatchblock closed 7 years ago

snatchblock commented 8 years ago

ProjectionExpression replaces the legacy AttributesToGet parameter.

snatchblock commented 8 years ago

sorry for the earlier PR confusion.. this is a proper PR / locally tested

adrianpraja commented 8 years ago

I checked your changes,

It's missing the "ExpressionAttributeNames"

ProjectionExpression and ExpressionAttributeNames should be transparent to the user not a method that they need to remember

Im working on something like:

DynamoDB
     .table()
     .select('MyList[0]', 'MyMap.nestedField')
     .query(...)

that uses ProjectionExpression and ExpressionAttributeNames in the back

I will try to to speed up and push my version sooner

Thanks