-
Notifications
You must be signed in to change notification settings - Fork 192
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
fix(v1): CPK has-many has-one associatedFields #2734
Conversation
061e8ed
to
88c4d67
Compare
764842d
to
06b6bf4
Compare
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## v1 #2734 +/- ##
==========================================
- Coverage 64.42% 64.28% -0.15%
==========================================
Files 720 720
Lines 22722 22736 +14
==========================================
- Hits 14639 14616 -23
- Misses 8083 8120 +37
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
06b6bf4
to
85dad80
Compare
Amplify/Categories/DataStore/Model/Internal/Schema/ModelSchema+Definition.swift
Outdated
Show resolved
Hide resolved
85dad80
to
1f3cc79
Compare
1f3cc79
to
fe29459
Compare
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.
LGTM
Issue #
Table of contents
data-dev-preview
fix: CPK uni-directional has-many lazy list load #2730v1
- fix(v1): CPK has-many has-one associatedFields #2734 (You are here)main
- fix: CPK has-many has-one associatedFields #2735Description
This PR does not attempt to fix the uni-directional has-many lazy loading list issue, see #2730 for more details and the fix. Since the CPK codegen changes (aws-amplify/amplify-codegen#538 ) are available in
v1
, this PR introduces that parameter to make sure newly generated types continue to compile with the addedassociatedWithFields
argument to hasMany model fields. When we are ready to move forward with the backport of #2711 then it will be solved onv1
.We are also fixing uni-directional has-one CPK use case by extending has-one with `associatedFields. Corresponding codegen changes are in aws-amplify/amplify-codegen#541
General Checklist
Given When Then
inline code documentation and are named accordinglytestThing_condition_expectation()
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.