Bugfix/required and nullable difference #150
Open
+86
−57
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi there,
We have found a problem concerning nullable fields which lead into more problems, if we are using
required
,readOnly
andnullabel
in combination.To keep the possibility as before,
strictRequiredChecks
has been added.undefined
valuesnull
valuesWe have implemented some changes:
nullable
andreadOnly
A field is required when:
readOnly
is false and property isrequired
is true orstrictRequiredChecks
is falseA field is nullable when:
nullable
is true orstrictNullChecks
is falseThese changes might break implementations of other developers, since
strictNullCheck
has been moved to thenullable
evaluation insteadrequired
akaundefined
.Minor changes and fixes:
Please let me know this poses a problem. Happy coding!