Add saved view home defaults

src/Exceptionless.Core/Models/Organization.cs

@@ -32,6 +32,12 @@ public Organization()
     [Required]
     public string Name { get; set; } = null!;
 
+    /// <summary>
+    /// The shared saved view used as the default landing view when a user has not selected a personal default.
+    /// </summary>
+    [ObjectId]
+    public string? DefaultSavedViewId { get; set; }
+
     [StringLength(2000)]
     public string? IconFileName { get; set; }
 

src/Exceptionless.Core/Models/User.cs

@@ -23,6 +23,7 @@ public record User : IIdentity, IHaveDates, IValidatableObject
     public string? PasswordResetToken { get; set; }
     public DateTime PasswordResetTokenExpiration { get; set; }
     public ICollection<OAuthAccount> OAuthAccounts { get; init; } = new Collection<OAuthAccount>();
+    public ICollection<UserOrganizationPreference> OrganizationPreferences { get; init; } = new Collection<UserOrganizationPreference>();
 
     /// <summary>
     /// Gets or sets the users Full Name.

src/Exceptionless.Core/Models/UserOrganizationPreference.cs

@@ -0,0 +1,13 @@
+using Exceptionless.Core.Attributes;
+using Foundatio.Repositories.Models;
+
+namespace Exceptionless.Core.Models;
+
+public sealed record UserOrganizationPreference
+{
+    [ObjectId]
+    public string OrganizationId { get; set; } = null!;
+
+    [ObjectId]
+    public string DefaultSavedViewId { get; set; } = null!;
+}

src/Exceptionless.Core/Services/OrganizationService.cs

@@ -93,6 +93,8 @@ public async Task<long> RemoveUsersAsync(Organization organization, string? curr
                 {
                     _logger.LogInformation("Removing user {User} from organization: {OrganizationName} ({Organization})", user.Id, organization.Name, organization.Id);
                     user.OrganizationIds.Remove(organization.Id);
+                    foreach (var preference in user.OrganizationPreferences.Where(preference => String.Equals(preference.OrganizationId, organization.Id, StringComparison.Ordinal)).ToList())
+                        user.OrganizationPreferences.Remove(preference);
                     usersToUpdate.Add(user);
                 }
             }

src/Exceptionless.Web/Api/Endpoints/SavedViewEndpoints.cs

@@ -71,6 +71,48 @@ public static IEndpointRouteBuilder MapSavedViewEndpoints(this IEndpointRouteBui
             }
         });
 
+        group.MapPut("organizations/{organizationId:objectid}/saved-view-defaults/user", async (string organizationId, IMediator mediator, IMediatorResultMapper<HttpIResult> resultMapper,
+            [FromBody] UpdateSavedViewDefault savedViewDefault)
+            => (await mediator.InvokeAsync<Result<UpdateSavedViewDefault>>(new SavedViewMessages.UpdateUserSavedViewDefault(organizationId, savedViewDefault))).ToHttpResult(resultMapper))
+        .Accepts<UpdateSavedViewDefault>("application/json", "application/*+json")
+        .Produces<UpdateSavedViewDefault>()
+        .ProducesProblem(StatusCodes.Status404NotFound)
+        .ProducesProblem(StatusCodes.Status422UnprocessableEntity)
+        .WithSummary("Update the current user's saved view default")
+        .WithMetadata(new EndpointDocumentation {
+            RequestBodyDescription = "The personal saved view default. A null saved view identifier clears the preference.",
+            RequestBodyRequired = true,
+            ParameterDescriptions = new() {
+                ["organizationId"] = "The identifier of the organization.",
+            },
+            ResponseDescriptions = new() {
+                ["200"] = "The personal saved view default was updated.",
+                ["404"] = "The organization could not be found.",
+                ["422"] = "The saved view is not accessible in this organization.",
+            }
+        });
+
+        group.MapPut("organizations/{organizationId:objectid}/saved-view-defaults/organization", async (string organizationId, IMediator mediator, IMediatorResultMapper<HttpIResult> resultMapper,
+            [FromBody] UpdateSavedViewDefault savedViewDefault)
+            => (await mediator.InvokeAsync<Result<UpdateSavedViewDefault>>(new SavedViewMessages.UpdateOrganizationSavedViewDefault(organizationId, savedViewDefault))).ToHttpResult(resultMapper))
+        .Accepts<UpdateSavedViewDefault>("application/json", "application/*+json")
+        .Produces<UpdateSavedViewDefault>()
+        .ProducesProblem(StatusCodes.Status404NotFound)
+        .ProducesProblem(StatusCodes.Status422UnprocessableEntity)
+        .WithSummary("Update the organization's saved view default")
+        .WithMetadata(new EndpointDocumentation {
+            RequestBodyDescription = "The shared saved view default. A null saved view identifier clears the preference.",
+            RequestBodyRequired = true,
+            ParameterDescriptions = new() {
+                ["organizationId"] = "The identifier of the organization.",
+            },
+            ResponseDescriptions = new() {
+                ["200"] = "The organization saved view default was updated.",
+                ["404"] = "The organization could not be found.",
+                ["422"] = "The saved view is private or is not accessible in this organization.",
+            }
+        });
+
         group.MapPost("organizations/{organizationId:objectid}/saved-views", async (string organizationId, IMediator mediator, IMediatorResultMapper<HttpIResult> resultMapper,
             [FromBody] NewSavedView savedView) =>
         {

src/Exceptionless.Web/Api/Handlers/OrganizationHandler.cs

@@ -704,6 +704,8 @@ public async Task<Result> Handle(RemoveOrganizationUser message)
             await organizationService.RemoveUserSavedViewsAsync(organization.Id, user.Id);
 
             user.OrganizationIds.Remove(organization.Id);
+            foreach (var preference in user.OrganizationPreferences.Where(preference => String.Equals(preference.OrganizationId, organization.Id, StringComparison.Ordinal)).ToList())
+                user.OrganizationPreferences.Remove(preference);
             await userRepository.SaveAsync(user, o => o.Cache());
             await messagePublisher.PublishAsync(new UserMembershipChanged
             {

src/Exceptionless.Web/Api/Handlers/SavedViewHandler.cs

@@ -26,6 +26,7 @@ namespace Exceptionless.Web.Api.Handlers;
 public partial class SavedViewHandler(
     ISavedViewRepository repository,
     IOrganizationRepository organizationRepository,
+    IUserRepository userRepository,
     ILockProvider lockProvider,
     IQueue<WorkItemData> workItemQueue,
     ApiMapper mapper,
@@ -82,6 +83,68 @@ public async Task<Result<ViewSavedView>> Handle(GetSavedViewById message)
         return MapToViewModel(model);
     }
 
+    public async Task<Result<UpdateSavedViewDefault>> Handle(UpdateUserSavedViewDefault message)
+    {
+        if (!HttpContext.Request.CanAccessOrganization(message.OrganizationId))
+            return Result.NotFound("Organization not found.");
+
+        if (await organizationRepository.GetByIdAsync(message.OrganizationId) is null)
+            return Result.NotFound("Organization not found.");
+
+        if (message.Default.SavedViewId is not null)
+        {
+            var savedView = await repository.GetByIdAsync(message.Default.SavedViewId, o => o.Cache(false));
+            if (savedView is null
+                || !String.Equals(savedView.OrganizationId, message.OrganizationId, StringComparison.Ordinal)
+                || (savedView.UserId is not null && !String.Equals(savedView.UserId, GetCurrentUserId(), StringComparison.Ordinal)))
+                return Result.Invalid(ValidationError.Create("saved_view_id", "The saved view is not accessible in this organization."));
+        }
+
+        var user = await userRepository.GetByIdAsync(GetCurrentUserId(), o => o.Cache(false));
+        if (user is null)
+            return Result.NotFound("User not found.");
+
+        foreach (var preference in user.OrganizationPreferences.Where(preference => String.Equals(preference.OrganizationId, message.OrganizationId, StringComparison.Ordinal)).ToList())
+            user.OrganizationPreferences.Remove(preference);
+
+        if (message.Default.SavedViewId is not null)
+        {
+            user.OrganizationPreferences.Add(new UserOrganizationPreference
+            {
+                OrganizationId = message.OrganizationId,
+                DefaultSavedViewId = message.Default.SavedViewId
+            });
+        }
+
+        await userRepository.SaveAsync(user, o => o.Cache());
+        return message.Default;
+    }
+
+    public async Task<Result<UpdateSavedViewDefault>> Handle(UpdateOrganizationSavedViewDefault message)
+    {
+        if (!HttpContext.Request.CanAccessOrganization(message.OrganizationId))
+            return Result.NotFound("Organization not found.");
+
+        var organization = await organizationRepository.GetByIdAsync(message.OrganizationId, o => o.Cache(false));
+        if (organization is null)
+            return Result.NotFound("Organization not found.");
+
+        if (message.Default.SavedViewId is not null)
+        {
+            var savedView = await repository.GetByIdAsync(message.Default.SavedViewId, o => o.Cache(false));
+            if (savedView is null
+                || !String.Equals(savedView.OrganizationId, message.OrganizationId, StringComparison.Ordinal)
+                || savedView.UserId is not null)
+            {
+                return Result.Invalid(ValidationError.Create("saved_view_id", "The organization default must be a shared saved view in this organization."));
+            }
+        }
+
+        organization.DefaultSavedViewId = message.Default.SavedViewId;
+        await organizationRepository.SaveAsync(organization, o => o.Cache().Consistency(Consistency.Immediate));
+        return message.Default;
+    }
+
     public async Task<Result<ViewSavedView>> Handle(CreateSavedView message)
     {
         if (!HttpContext.Request.IsInOrganization(message.OrganizationId))
@@ -258,6 +321,7 @@ public async Task<Result<ModelActionResults>> Handle(DeleteSavedViews message)
         if (deletableItems.Count == 0)
             return results.Failure.Count == 1 ? Result<ModelActionResults>.FromResult(PermissionToResult(results.Failure.First())) : results;
 
+        await ClearDefaultReferencesAsync(deletableItems);
         await repository.RemoveAsync(deletableItems);
 
         if (results.Failure.Count == 0)
@@ -454,6 +518,21 @@ private ViewSavedView MapToViewModel(SavedView model)
 
     private List<ViewSavedView> MapToViewModels(IEnumerable<SavedView> models) => models.Select(MapToViewModel).ToList();
 
+    private async Task ClearDefaultReferencesAsync(IReadOnlyCollection<SavedView> deletedSavedViews)
+    {
+        var deletedIds = deletedSavedViews.Select(savedView => savedView.Id).ToHashSet(StringComparer.Ordinal);
+
+        foreach (string organizationId in deletedSavedViews.Select(savedView => savedView.OrganizationId).Distinct(StringComparer.Ordinal))
+        {
+            var organization = await organizationRepository.GetByIdAsync(organizationId);
+            if (organization?.DefaultSavedViewId is null || !deletedIds.Contains(organization.DefaultSavedViewId))
+                continue;
+
+            organization.DefaultSavedViewId = null;
+            await organizationRepository.SaveAsync(organization, o => o.Cache().Consistency(Consistency.Immediate));
+        }
+    }
+
     private string GetCurrentUserId() => HttpContext.Request.GetUser().Id;
 
     private static void AfterResultMap<TDestination>(ICollection<TDestination> models)

src/Exceptionless.Web/Api/Messages/SavedViewMessages.cs

@@ -7,6 +7,8 @@ namespace Exceptionless.Web.Api.Messages;
 public record GetSavedViewsByOrganization(string OrganizationId, int Page, int Limit);
 public record GetSavedViewsByView(string OrganizationId, string ViewType, int Page, int Limit);
 public record GetSavedViewById(string Id);
+public record UpdateUserSavedViewDefault(string OrganizationId, UpdateSavedViewDefault Default);
+public record UpdateOrganizationSavedViewDefault(string OrganizationId, UpdateSavedViewDefault Default);
 public record CreateSavedView(string OrganizationId, NewSavedView SavedView);
 public record CreatePredefinedSavedViews(string OrganizationId);
 public record GetPredefinedSavedViews;

src/Exceptionless.Web/ClientApp/e2e/tests/authentication.e2e.ts

@@ -21,15 +21,15 @@ test('user can recover from a failed login, restore the session, and log out', a
             await page.getByPlaceholder('Enter password').fill(E2E_TEST_PASSWORD);
             await page.getByRole('button', { exact: true, name: 'Login' }).click();
 
-            await expect(page.getByRole('heading', { name: 'Stacks' })).toBeVisible({ timeout: 30_000 });
-            await expect(page).toHaveURL(/\/next\/stack(?:[?#]|$)/);
+            await expect(page.getByRole('heading', { name: 'All' })).toBeVisible({ timeout: 30_000 });
+            await expect(page).toHaveURL(/\/next\/stack\/all(?:[?#]|$)/);
         });
 
         await test.step('restore the authenticated application after a reload', async () => {
             await page.reload();
 
-            await expect(page.getByRole('heading', { name: 'Stacks' })).toBeVisible({ timeout: 30_000 });
-            await expect(page).toHaveURL(/\/next\/stack(?:[?#]|$)/);
+            await expect(page.getByRole('heading', { name: 'All' })).toBeVisible({ timeout: 30_000 });
+            await expect(page).toHaveURL(/\/next\/stack\/all(?:[?#]|$)/);
         });
 
         await test.step('log out through the user menu', async () => {

src/Exceptionless.Web/ClientApp/e2e/tests/exie-full-page.e2e.ts

@@ -53,7 +53,7 @@ test('Exie opens from navigation and expands without losing the conversation', a
 
     await test.step('expand the side panel and retain its conversation and source URL', async () => {
         await page.getByRole('link', { name: 'Collapse Exie to side panel' }).click();
-        await expect(page).toHaveURL(/\/next\/stack(?:[?#]|$)/);
+        await expect(page).toHaveURL(/\/next\/stack\/all(?:[?#]|$)/);
         await expect(page.locator('[data-assistant-panel]')).toBeVisible();
         await page.getByRole('button', { name: 'Close Exie' }).click();
 

src/Exceptionless.Web/ClientApp/e2e/tests/password-recovery.e2e.ts

@@ -34,6 +34,7 @@ test('user can reset a forgotten password and log in @signup', async ({ e2eApi,
         await page.getByPlaceholder('Enter password').fill(RESET_PASSWORD);
         await page.getByRole('button', { exact: true, name: 'Login' }).click();
 
-        await expect(page.getByRole('heading', { name: 'Stacks' })).toBeVisible({ timeout: 30_000 });
+        await expect(page.getByRole('heading', { name: 'All' })).toBeVisible({ timeout: 30_000 });
+        await expect(page).toHaveURL(/\/next\/stack\/all(?:[?#]|$)/);
     });
 });

src/Exceptionless.Web/ClientApp/e2e/tests/saved-views.e2e.ts

@@ -4,6 +4,97 @@ import { expect, test } from '../fixtures/e2e-test';
 import { ExceptionlessE2EJourney } from '../support/exceptionless-journey';
 import { getVisibleText } from '../support/page-helpers';
 
+test('home navigation honors personal and organization saved views and survives deletion', async ({ e2eApi, e2eScenario, page, request }) => {
+    const failedApiRequests = captureFailedApiRequests(page);
+    const savedViewListLimits: string[] = [];
+    page.on('request', (request) => {
+        const url = new URL(request.url());
+        if (url.pathname === `/api/v2/organizations/${e2eScenario.organizationId}/saved-views`) {
+            savedViewListLimits.push(url.searchParams.get('limit') ?? '');
+        }
+    });
+    const journey = ExceptionlessE2EJourney.fromScenario(page, e2eApi, e2eScenario);
+    const viewName = `E2E Home ${journey.run.slice(-36)}`;
+    const viewSlug = savedViewSlug(viewName);
+
+    await test.step('fall back to the first Stacks saved view when no default is configured', async () => {
+        await page.goto('/next/');
+        await expect(page).toHaveURL(/\/next\/stack\/all(?:[?#]|$)/);
+        await expect(page.getByRole('heading', { name: 'All' })).toBeVisible({ timeout: 30_000 });
+        await expect.poll(() => savedViewListLimits).toContain('100');
+    });
+
+    await test.step('prefer the personal saved view', async () => {
+        await journey.submitRepresentativeEvent();
+        await page.goto(`/next/event?reference=${encodeURIComponent(journey.referenceId)}&time=all`);
+        await expect(getVisibleText(page, journey.message)).toBeVisible({ timeout: 30_000 });
+
+        await openViewMenu(page);
+        await page.getByRole('menuitem', { name: 'Save As...' }).click();
+        const dialog = page.getByRole('dialog', { name: 'Save View' });
+        await dialog.getByLabel('Name', { exact: true }).fill(viewName);
+        await dialog.getByRole('button', { name: 'Save' }).click();
+        await expect(dialog).toBeHidden({ timeout: 30_000 });
+        await expect(page.getByRole('heading', { name: viewName })).toBeVisible({ timeout: 30_000 });
+
+        await openViewMenu(page);
+        await page.getByRole('menuitem', { name: 'Set as my home view' }).click();
+        await expect(page.getByText(`"${viewName}" is now your home view.`)).toBeVisible();
+
+        await expect
+            .poll(
+                async () => {
+                    const response = await request.get(`/api/v2/organizations/${e2eScenario.organizationId}/saved-views/events`, {
+                        headers: { Authorization: `Bearer ${e2eScenario.userToken}` }
+                    });
+                    const savedViews = response.ok() ? ((await response.json()) as { name: string }[]) : [];
+                    return savedViews.some((savedView) => savedView.name === viewName);
+                },
+                { timeout: 30_000 }
+            )
+            .toBe(true);
+
+        await page.goto('/next/');
+        await expect(page).toHaveURL(new RegExp(`/next/event/${escapeRegExp(viewSlug)}(?:[?#]|$)`));
+    });
+
+    await test.step('fall back to the organization saved view after clearing the personal preference', async () => {
+        await openViewMenu(page);
+        await page.getByRole('menuitem', { name: 'Set as organization home' }).click();
+        await expect(page.getByText(`"${viewName}" is now the organization home view.`)).toBeVisible();
+
+        await openViewMenu(page);
+        await page.getByRole('menuitem', { name: 'Clear my home view' }).click();
+        await expect(page.getByText('Personal home view cleared.')).toBeVisible();
+
+        await page.goto('/next/');
+        await expect(page).toHaveURL(new RegExp(`/next/event/${escapeRegExp(viewSlug)}(?:[?#]|$)`));
+    });
+
+    await test.step('clear deleted defaults and return to the first Stacks saved view', async () => {
+        const deletion = await page.evaluate(
+            async ({ organizationId, token, viewName }) => {
+                const headers = { Authorization: `Bearer ${token}` };
+                const savedViewsResponse = await fetch(`/api/v2/organizations/${organizationId}/saved-views/events`, { headers });
+                const savedViews = await savedViewsResponse.json();
+                const savedView = savedViews.find((view: { name: string }) => view.name === viewName);
+                const response = await fetch(`/api/v2/saved-views/${savedView.id}`, {
+                    headers,
+                    method: 'DELETE'
+                });
+                return response.status;
+            },
+            { organizationId: e2eScenario.organizationId, token: e2eScenario.userToken, viewName }
+        );
+        expect(deletion).toBe(202);
+
+        await page.goto('/next/');
+        await expect(page).toHaveURL(/\/next\/stack\/all(?:[?#]|$)/);
+    });
+
+    expect(failedApiRequests).toEqual([]);
+});
+
 test('events saved view can be saved, renamed, loaded, and deleted', async ({ e2eApi, e2eScenario, page, request }) => {
     const failedApiRequests = captureFailedApiRequests(page);
     const journey = ExceptionlessE2EJourney.fromScenario(page, e2eApi, e2eScenario);