aj/MoneyMap
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: remove unnecessary async from OnPostRejectProposalAsync methods

Browse Source 
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.
This commit is contained in:
AJ
2025-11-16 11:53:36 -05:00
parent ea14466f57
commit f9cf709206
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
MoneyMap/Pages/ReviewAISuggestions.cshtml.cs
View File

@@ -99,7 +99,7 @@ public class ReviewAISuggestionsModel : PageModel
return RedirectToPage(); return RedirectToPage();
} }
public async Task<IActionResult> OnPostRejectProposalAsync(long transactionId) public IActionResult OnPostRejectProposalAsync(long transactionId)
{ {
// Just refresh the page, removing this transaction from view // Just refresh the page, removing this transaction from view
SuccessMessage = "Suggestion rejected."; SuccessMessage = "Suggestion rejected.";

2
MoneyMap/Pages/ReviewAISuggestionsWithProposals.cshtml.cs
View File

@@ -100,7 +100,7 @@ public class ReviewAISuggestionsWithProposalsModel : PageModel
return RedirectToPage(); return RedirectToPage();
} }
public async Task<IActionResult> OnPostRejectProposalAsync(long transactionId) public IActionResult OnPostRejectProposalAsync(long transactionId)
{ {
// Remove this proposal from session // Remove this proposal from session
var proposalsJson = HttpContext.Session.GetString("AIProposals"); var proposalsJson = HttpContext.Session.GetString("AIProposals");