Skip to content

Commit 5cdc49f

Browse files
Fix "View all notifications" link navigating to preferences instead of notifications list (#165)
* Initial plan * Fix 'View all notifications' link to point to AllNotifications action Co-authored-by: frasermolyneux <34033625+frasermolyneux@users.noreply.github.com> * Fix pre-existing IDE0390 build error in BannersController Co-authored-by: frasermolyneux <34033625+frasermolyneux@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: frasermolyneux <34033625+frasermolyneux@users.noreply.github.com>
1 parent 7f1dc34 commit 5cdc49f

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/XtremeIdiots.Portal.Web/Controllers/BannersController.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public class BannersController(
2929
public async Task<IActionResult> GameServersList(CancellationToken cancellationToken = default)
3030
{
3131
return await ExecuteWithErrorHandlingAsync(
32-
async () => View(),
32+
() => Task.FromResult<IActionResult>(View()),
3333
"Display game servers list view for banner management").ConfigureAwait(false);
3434
}
3535
}

src/XtremeIdiots.Portal.Web/Views/Shared/_TopNavbar.cshtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
</div>
4242
</div>
4343
<div class="border-top px-3 py-2 text-center">
44-
<a href="/Profile/Notifications" class="text-decoration-none small">View all notifications</a>
44+
<a asp-controller="Profile" asp-action="AllNotifications" class="text-decoration-none small">View all notifications</a>
4545
</div>
4646
</div>
4747
</li>

0 commit comments

Comments
 (0)