Skip to content

organization-layout-areas#84

Closed
sierragolflima wants to merge 1 commit into
mainfrom
organization-layout-areas
Closed

organization-layout-areas#84
sierragolflima wants to merge 1 commit into
mainfrom
organization-layout-areas

Conversation

@sierragolflima
Copy link
Copy Markdown
Contributor

@sierragolflima sierragolflima commented Apr 14, 2026

Changes to Layout Areas

Why

Todo board views (e.g. /ACME/CustomerOnboarding/TodosByCategory) broke in the portal because the security layer couldn't tell the difference between a user requesting data and a hub requesting data from another hub during initialization. It treated the hub like an unknown user and denied access, causing every child Todo card to fail to load.

How

  • Mark hub-to-hub communication as trusted: Added an IsHub flag to AccessContext so the security pipeline can recognize internal hub requests and let them through without applying user-level permission checks. This flag is only set by internal infrastructure code, never by user-facing paths.
  • Aligned test configuration with the portal: The existing test was missing several services the portal uses (row-level security, AI, threads, activity tracking, API tokens). Updated TodoBoardThumbnailTest to mirror the portal's full configuration so these kinds of failures get caught in CI.

@sierragolflima sierragolflima changed the title Changes to layout areas organization-layout-areas Apr 14, 2026
@github-actions
Copy link
Copy Markdown

Test Results

2 568 tests  ±0   2 555 ✅ ±0   5m 28s ⏱️ +17s
   31 suites ±0      13 💤 ±0 
   31 files   ±0       0 ❌ ±0 

Results for commit 446b208. ± Comparison against base commit 1df5249.

@sierragolflima
Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

@sierragolflima
Copy link
Copy Markdown
Contributor Author

Superseded by #94 (merged). Root cause was wrong NodeType on sample orgs — simple one-line fix per file.

@sierragolflima sierragolflima deleted the organization-layout-areas branch April 20, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant