-
-
Notifications
You must be signed in to change notification settings - Fork 360
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[v3] Model represents entity #1219
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/dynamoose/dynamoose/B34UekJ35vwT1wXUVPNY6XsEMKFk [Deployment for a75b468 failed] |
This pull request is stale because it has been open 7 days with no activity. Remove stale label or comment or this will be closed in 3 days. |
This pull request is stale because it has been open 7 days with no activity. Remove stale label or comment or this will be closed in 3 days. |
Next steps:
|
# Conflicts: # lib/Model/index.ts # package-lock.json # package.json # test/types/Model.ts # test/unit/Model.js
# Conflicts: # lib/Model/index.ts # test/unit/Model.js
Summary:
This PR changes the behavior of
Model
to represent an entity and adds the concept ofTable
.Type (select 1):
Is this a breaking change? (select 1):
Is this ready to be merged into Dynamoose? (select 1):
Are all the tests currently passing on this PR? (select 1):
Other:
npm test
npm run lint