search

peak-ai / jedlik

DynamoDB ODM for Node
MIT License
106 stars 4 forks source link

Expose the Database class as part of the top level API #15

Closed harrim91 closed 4 years ago

harrim91 commented 4 years ago

Is your feature request related to a problem? Please describe. I think the Database class would be really useful in its own right, without all of the Model/Document stuff. It would just be like a better DocumentClient. We should expose it as part of the top level API.

If this does happen, I'd like to change the name of the class as well - maybe to Client?

const client = new jedlik.Client({ tableName: 'users' });

client.scan({ limit: 5 }); // returns plain objects, not documents

client.put({ name: 'Michael' }); // doesn't do validation or anything like that

// etc
issue-label-bot[bot] commented 4 years ago

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.97. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!

Links: app homepage, dashboard and code for this bot.