search

clarkie / dynogels

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

Add secondary index scan #116

Open C45tr0 opened 6 years ago

C45tr0 commented 6 years ago

Updating #45. Let me know if this is the wrong way to handle this.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.003%) to 98.664% when pulling cfd3a133fab1aafea2ccfced848da6047a7ebbb7 on C45tr0:add-secondary-index-scan into f0b023d5d635b2b8fc3ed8cf0612878ea21286ce on clarkie:master.

jkav77 commented 6 years ago

@C45tr0 I made a slight modification to the documentation by pulling the Indexes section out of the Query section since it now applies to Query and Scan. Is this good to go in your opinion?

clarkie commented 6 years ago

There were conflicts, I tried to resolve them but the tests are still failing. I'll have a look over coffee in the morning.