From f9cf70920670a07ef8a43b7324b00b6116c7f385 Mon Sep 17 00:00:00 2001 From: AJ Date: Sun, 16 Nov 2025 11:53:36 -0500 Subject: [PATCH] Fix: remove unnecessary async from OnPostRejectProposalAsync methods Removed async/await keywords from methods that don't perform any async operations. The methods were triggering CS1998 warnings and running synchronously despite being marked async. --- MoneyMap/Pages/ReviewAISuggestions.cshtml.cs | 2 +- MoneyMap/Pages/ReviewAISuggestionsWithProposals.cshtml.cs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/MoneyMap/Pages/ReviewAISuggestions.cshtml.cs b/MoneyMap/Pages/ReviewAISuggestions.cshtml.cs index dab80c8..8c94f8e 100644 --- a/MoneyMap/Pages/ReviewAISuggestions.cshtml.cs +++ b/MoneyMap/Pages/ReviewAISuggestions.cshtml.cs @@ -99,7 +99,7 @@ public class ReviewAISuggestionsModel : PageModel return RedirectToPage(); } - public async Task OnPostRejectProposalAsync(long transactionId) + public IActionResult OnPostRejectProposalAsync(long transactionId) { // Just refresh the page, removing this transaction from view SuccessMessage = "Suggestion rejected."; diff --git a/MoneyMap/Pages/ReviewAISuggestionsWithProposals.cshtml.cs b/MoneyMap/Pages/ReviewAISuggestionsWithProposals.cshtml.cs index 8d380b4..15cb11d 100644 --- a/MoneyMap/Pages/ReviewAISuggestionsWithProposals.cshtml.cs +++ b/MoneyMap/Pages/ReviewAISuggestionsWithProposals.cshtml.cs @@ -100,7 +100,7 @@ public class ReviewAISuggestionsWithProposalsModel : PageModel return RedirectToPage(); } - public async Task OnPostRejectProposalAsync(long transactionId) + public IActionResult OnPostRejectProposalAsync(long transactionId) { // Remove this proposal from session var proposalsJson = HttpContext.Session.GetString("AIProposals");