From 13d2b3c0ff31c4004b20e0d135f9a5aab90816c9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Roland=20M=C3=A1rkus?= Date: Sun, 16 Jun 2024 21:46:15 +0200 Subject: [PATCH] Adding explaining comment to avoid removing SaveAsync() --- .../Services/EmailQuotaService.cs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Lombiq.Hosting.Tenants.EmailQuotaManagement/Services/EmailQuotaService.cs b/Lombiq.Hosting.Tenants.EmailQuotaManagement/Services/EmailQuotaService.cs index 92af9ffb..e3566e4a 100644 --- a/Lombiq.Hosting.Tenants.EmailQuotaManagement/Services/EmailQuotaService.cs +++ b/Lombiq.Hosting.Tenants.EmailQuotaManagement/Services/EmailQuotaService.cs @@ -19,6 +19,8 @@ namespace Lombiq.Hosting.Tenants.EmailQuotaManagement.Services; +// As noted under github issue https://github.com/OrchardCMS/OrchardCore/issues/15290#issuecomment-2093363619 +// SaveAsync calls should be removed due to redundancy, however removing them breaks functionality. public class EmailQuotaService : IEmailQuotaService { private readonly ISession _session;