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

Simplified notification updating #30

Merged
khwezi merged 1 commits from emailjobs into master 2026-05-17 16:01:25 +02:00
Conversation 0 Commits 1 Files Changed 2
+3 -7
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() 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), To = DateOnly.FromDateTime(DateTime.UtcNow),
MaxRecords = 10 MaxRecords = 10
}; };
LiteCharms.Features/Shop/Notifications/NotificationService.cs
-4
View File
@@ -97,12 +97,8 @@ public class NotificationService(IDbContextFactory<ShopDbContext> contextFactory
notification.Processed = request.Processed; notification.Processed = request.Processed;
notification.UpdatedAt = DateTime.UtcNow; notification.UpdatedAt = DateTime.UtcNow;
if (request.HasError)
{
notification.HasError = request.HasError; notification.HasError = request.HasError;
notification.Errors = request.Errors; notification.Errors = request.Errors;
}
return await context.SaveChangesAsync(cancellationToken) > 0 return await context.SaveChangesAsync(cancellationToken) > 0
? Result.Ok() ? Result.Ok()