litecharms/components
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 114 Wiki Activity

Simplified notification updating
continuous-integration/drone/pr Build is passing
Details

Browse Source
This commit is contained in:
Khwezi Mngoma
2026-05-17 16:00:35 +02:00
parent 97bde73777
commit da141311ff
2 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
LiteCharms.Features.Tests/NotificationsFeatureTests.cs
+1 -1
View File
@@ -40,7 +40,7 @@ public class NotificationsFeatureTests(CommonFixture fixture, ITestOutputHelper
{
DateRange range = new()
{
From = DateOnly.FromDateTime(new DateTime(2026, 05, 01, 0, 0, 0, DateTimeKind.Utc)),
From = DateOnly.FromDateTime(new DateTime(2026, 04, 01, 0, 0, 0, DateTimeKind.Utc)),
To = DateOnly.FromDateTime(DateTime.UtcNow),
MaxRecords = 10
};
LiteCharms.Features/Shop/Notifications/NotificationService.cs
+2 -6
View File
@@ -97,12 +97,8 @@ public class NotificationService(IDbContextFactory<ShopDbContext> contextFactory
notification.Processed = request.Processed;
notification.UpdatedAt = DateTime.UtcNow;
if (request.HasError)
{
notification.HasError = request.HasError;
notification.Errors = request.Errors;
}
notification.HasError = request.HasError;
notification.Errors = request.Errors;
return await context.SaveChangesAsync(cancellationToken) > 0
? Result.Ok()