search

clarkie / dynogels

DynamoDB data mapper for node.js. Originally forked from https://github.com/ryanfitz/vogels
Other
490 stars 110 forks source link

Added the ability for the expressions api to support document paths #20

Closed C45tr0 closed 7 years ago

C45tr0 commented 8 years ago

Added ability to handle Map.Attr document path.

This is only for the Map, but the other ones should probably be added also. http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/Expressions.AccessingItemAttributes.html#DocumentPaths

This is pulled from my change to vogels.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.001%) to 98.629% when pulling a2d69a6da9f1d60d1ac65ea3a5418343f23e0f55 on C45tr0:expressions-document-paths into a7842fe19252bd4b596c21bb4cdafa4344667950 on clarkie:master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.001%) to 98.629% when pulling a2d69a6da9f1d60d1ac65ea3a5418343f23e0f55 on C45tr0:expressions-document-paths into a7842fe19252bd4b596c21bb4cdafa4344667950 on clarkie:master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.003%) to 98.631% when pulling 01409dee0e9f1d947ffcfb51ec0c2229df13e1be on C45tr0:expressions-document-paths into a7842fe19252bd4b596c21bb4cdafa4344667950 on clarkie:master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.003%) to 98.631% when pulling 01409dee0e9f1d947ffcfb51ec0c2229df13e1be on C45tr0:expressions-document-paths into a7842fe19252bd4b596c21bb4cdafa4344667950 on clarkie:master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.003%) to 98.631% when pulling 01409dee0e9f1d947ffcfb51ec0c2229df13e1be on C45tr0:expressions-document-paths into a7842fe19252bd4b596c21bb4cdafa4344667950 on clarkie:master.

clarkie commented 8 years ago

@C45tr0 thanks for this. I think it'd be good to add this use case to the integration tests and also update the docs.

C45tr0 commented 8 years ago

Good call. I will look into that today hopefully.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.003%) to 98.662% when pulling dcc2de2deb755fdf6d04a0c2e64383c7c63f8757 on C45tr0:expressions-document-paths into 0f1961c292a7c64641f5a2f7d745ac77552952ca on clarkie:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.003%) to 98.662% when pulling dcc2de2deb755fdf6d04a0c2e64383c7c63f8757 on C45tr0:expressions-document-paths into 0f1961c292a7c64641f5a2f7d745ac77552952ca on clarkie:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.003%) to 98.662% when pulling 73a9ee9bb81201e271dbdea327770ba9ab89c890 on C45tr0:expressions-document-paths into 0f1961c292a7c64641f5a2f7d745ac77552952ca on clarkie:master.