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

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

Browse Source 
Reviewed-on: #30
This commit was merged in pull request #30.
This commit is contained in:
Khwezi Mngoma
2026-05-17 16:01:24 +02:00
parent ab3d8e6e9a da141311ff
commit 7743c3178e
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()