Nullable Reference types#85
Merged
Merged
Conversation
Updated several interface files in the `E13.Common.Domain` namespace to change properties from non-nullable types to nullable types. This allows properties like `DeletedBy`, `EffectiveBy`, and others to hold null values, enabling better representation of optional data or states within the entities.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This pull request introduces nullable reference types for several properties in domain interfaces to enhance compatibility with scenarios where the properties might not always have a value. The changes primarily focus on making the
Ttype properties nullable in five interfaces.Nullable reference type adjustments:
src/E13.Common.Domain/IDeletable.cs: Updated theDeletedByproperty to be nullable (T?) to allow for cases where no deletion user is specified.src/E13.Common.Domain/IEffectable.cs: Updated theEffectiveByproperty to be nullable (T?) to accommodate scenarios where no effective user is provided.src/E13.Common.Domain/IExpirable.cs: Updated theExpirationByproperty to be nullable (T?) to handle situations where no expiration user is defined.src/E13.Common.Domain/IModifiable.cs: Updated theModifiedByproperty to be nullable (T?) to support cases where no modification user is specified.src/E13.Common.Domain/IOwnable.cs: Updated theOwnedByproperty to be nullable (T?) to allow for ownership to be unspecified.