generated from Avanade/avanade-template
-
Notifications
You must be signed in to change notification settings - Fork 7
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.31.0 #136
Conversation
This file contains 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
… `CosmosDbContainer` and `CosmosDbModelContainer` are model type independent, with underlying type support implemented at the method level for greater flexibility and control. The typed `CosmosDbContainer<T, TModel>` etc. remain and are accessed from the type independent containers as required. - The existing `IMultiSetArgs` operations have been moved (and renamed) from `CosmosDb` to `CosmosDbContainer` and `CosmosDbModelContainer` as these are single container-specific. - The existing `CosmosDb.UseAuthorizeFilter` operations have been moved to `CosmosDbContainer` as these are single container-specific.
…date` to encapsulate `ChangeLog.PrepareCreated` and `ChangeLog.PrepareUpdated`, and `Cleaner.ResetTenantId` to ensure consistent handling throughout _CoreEx_. - *Enhancement:* Added `SystemTime.Timestamp` as the standard means to access the current timestamp (uses `Cleaner.Clean`) to ensure consistency throughout _CoreEx_. Therefore, the likes of `DateTime.UtcNow` should be replaced with `SystemTime.Timestamp`. The previous `ExecutionContent.SystemTime` has been removed as it was not consistent with the `ExecutionContext` pattern.
…ementations to check `SettingsBase` configuration to enable/disable.
- Update packages
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
JsonMergePatch
toExtended.JsonMergePatchEx
; this remains theAddJsonMergePatch
default implementation.JsonMergePatch
that leveragesJsonElement
andUtf8JsonWriter
without underlying reflection; useful in scenarios where the value type is not known. This is also not as performant as the reflection-basedJsonMergePatchEx
version and the primary reason why it is not the new default.CosmosDb
capabilities such that theCosmosDbContainer
andCosmosDbModelContainer
are model type independent, with underlying type support implemented at the method level for greater flexibility and control. The typedCosmosDbContainer<T, TModel>
etc. remain and are accessed from the type independent containers as required.IMultiSetArgs
operations have been moved (and renamed) fromCosmosDb
toCosmosDbContainer
andCosmosDbModelContainer
as these are single container-specific.CosmosDb.UseAuthorizeFilter
operations have been moved toCosmosDbContainer
as these are single container-specific.Cleaner.PrepareCreate
andCleaner.PrepareUpdate
to encapsulateChangeLog.PrepareCreated
andChangeLog.PrepareUpdated
, andCleaner.ResetTenantId
to ensure consistent handling throughout CoreEx.SystemTime.Timestamp
as the standard means to access the current timestamp (usesCleaner.Clean
) to ensure consistency throughout CoreEx. Therefore, the likes ofDateTime.UtcNow
should be replaced withSystemTime.Timestamp
. The previousExecutionContent.SystemTime
has been removed as it was not consistent with theExecutionContext
pattern.IExtendedException.ShouldBeLogged
implementations to checkSettingsBase
configuration to enable/disable.