Merge pull request 'Simplified notification updating' (#30) from emailjobs into master

Reviewed-on: #30

LiteCharms.Features.Tests/NotificationsFeatureTests.cs

@@ -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

@@ -97,12 +97,8 @@ public class NotificationService(IDbContextFactory<ShopDbContext> contextFactory
 
             notification.Processed = request.Processed;
             notification.UpdatedAt = DateTime.UtcNow;
-
+            notification.HasError = request.HasError;
-            if (request.HasError)
+            notification.Errors = request.Errors;
-            {
-                notification.HasError = request.HasError;
-                notification.Errors = request.Errors;
-            }
 
             return await context.SaveChangesAsync(cancellationToken) > 0
                 ? Result.Ok()