feat(mosaic): wire enterprise accounts - #9973
Conversation
🦋 Changeset detectedLatest commit: be39204 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (6)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: This review used your included allowance. 6 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 10 reviews per hour. 📝 WalkthroughWalkthroughAdds enterprise account projection and connection flows to the user-profile feature. The section displays active enterprise accounts and eligible unlinked connections, and supports creating an external account and navigating to its verification redirect. The change adds connection pending and error states, exposes a profile-panel slot, and adds a live enterprise accounts page and sidebar link. It also adds FAPI fixtures and tests for projection, connection behavior, and integration. Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~25 minutes Merge Risk: 🟡 Moderate · up to Users with linked OAuth enterprise accounts can still be offered those same connections. Correct linkage matching before merging, or explicitly accept this limitation. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Comment |
1410679 to
d11c377
Compare
d36fd36 to
bb36d55
Compare
d11c377 to
29bc6c5
Compare
bb36d55 to
6f480af
Compare
48651b4 to
a652e2c
Compare
960966e to
be39204
Compare
Description
Wire Mosaic's enterprise accounts section to Clerk so users can see linked accounts and connect eligible organization SSO connections. Connect calls the resource API directly and displays failures through the existing error UI. Session reverification UI is deferred. Add the profile-panel slot, section styles, and a Swingset live page at
/live/enterprise-accounts.Staging verification exposed an existing backend issue: SAML linking silently fails when the IdP email matches the user's already-verified primary email. The callback attempts to finalize that email's verification again and rejects it as already existing. Linking with a different email succeeds. That backend fix is outside this PR.
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change