-
Notifications
You must be signed in to change notification settings - Fork 247
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
feat(datastore): implement custom primary key functionality #1517
feat(datastore): implement custom primary key functionality #1517
Conversation
fde239a
to
b968355
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feat/custom-pk #1517 +/- ##
==================================================
- Coverage 46.32% 36.34% -9.99%
==================================================
Files 263 244 -19
Lines 10229 8191 -2038
==================================================
- Hits 4739 2977 -1762
+ Misses 5490 5214 -276
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall. Maybe I missed it - are there unit tests for this?
@@ -19,661 +19,664 @@ import com.amplifyframework.core.model.* | |||
import com.amplifyframework.core.model.temporal.Temporal | |||
|
|||
val postSchema = ModelSchema.builder() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do any of these need to be updated with indexes? Or should we add one with indexes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The test model Post
has a SDI defined, the index info has been added to this testing model and its schema, see line 24 - 26. And it's asserted by unit tests. iOS impl. has similar changes too.
…ards compatibility
- FlutterModelPrimaryKey implementation - Query by identifier (ModelPrimaryKey) using new amplify-android API - Required code changes to fit the newly updated interface in amplify-android - Fixing exiting unit tests
Issue #, if available:
#1426
Description of changes:
Create native objects presenting model indexes and let native storage engine create data tables with expected indexes.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.