Skip to content

Fix organization context when following resource links - #2565

Merged
ejsmith merged 3 commits into
mainfrom
issue/resource-organization-navigation
Sep 8, 2026
Merged

Fix organization context when following resource links#2565
ejsmith merged 3 commits into
mainfrom
issue/resource-organization-navigation

Conversation

@ejsmith

@ejsmith ejsmith commented Sep 8, 2026

Copy link
Copy Markdown
Member

Links to another accessible organization previously redirected to the selected organization, while stack and event detail links could retain the wrong organization context. Select the organization from the successful resource lookup and preserve the requested destination, including project and invoice links. Nested project/stack URLs redirect to the stack’s canonical page when its owning project differs.

Keep organization Users and Billing scoped to the route, and preserve the current selection when access is denied. Public API contracts and authorization are unchanged.

Validation:

  • npm run validate
  • 17 Playwright cases covering resource links, the Exie usage table, history/reload, sidebar switching, ordinary members, and denied access
  • 13 focused Vitest tests for organization APIs and event navigation

Exie usage rows, invoice data, empty-stack responses, and access-denial responses use controlled browser fixtures; resource records and event ingestion use the local Aspire stack.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-08T04:34:36.745773Z 294d40f New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@ejsmith
ejsmith force-pushed the issue/resource-organization-navigation branch from d7c6fea to 0050f9f Compare September 8, 2026 03:37

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0050f9fc52

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ejsmith

ejsmith commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

/preview

@github-actions github-actions Bot added the dev-preview Deploy this pull request to the shared dev environment. label Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Preview deployed

@ejsmith
ejsmith merged commit 994df66 into main Sep 8, 2026
7 checks passed
@ejsmith
ejsmith deleted the issue/resource-organization-navigation branch September 8, 2026 04:32
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Complexity Health
Exceptionless.Insulation 37% 35% 286
Exceptionless.Web 85% 70% 8102
Exceptionless.Core 76% 68% 10497
Exceptionless.AppHost 38% 41% 147
Summary 79% (26318 / 33338) 68% (12215 / 18003) 19032

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-preview Deploy this pull request to the shared dev environment.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant