Skip to content

feat: a user can access apis within workspace context#3775

Merged
perkinsjr merged 45 commits intoslug-routingfrom
eng-1893-a-user-can-access-apis-within-workspace-context
Sep 11, 2025
Merged

feat: a user can access apis within workspace context#3775
perkinsjr merged 45 commits intoslug-routingfrom
eng-1893-a-user-can-access-apis-within-workspace-context

Conversation

@MichaelUnkey
Copy link
Collaborator

@MichaelUnkey MichaelUnkey commented Aug 11, 2025

What does this PR do?

Fixes # (issue)
Eng 1893
If there is not an issue for this, please create one first. This is used to tracking purposes and also helps use understand why this PR exists

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • Enhancement (small improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How should this be tested?

  • all /apis routes are now /[workspaceId]/apis routes

Checklist

Required

  • Filled out the "How to test" section in this PR
  • Read Contributing Guide
  • Self-reviewed my own code
  • Commented on my code in hard-to-understand areas
  • Ran pnpm build
  • Ran pnpm fmt
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the Unkey Docs if changes were necessary

Summary by CodeRabbit

  • New Features

    • Workspace-aware routing across the dashboard: all API, Keys, Settings, and Logs pages now use workspace-scoped URLs and breadcrumbs.
    • Sidebar navigation and API lists reflect the active workspace; deep links (including Identities and Authorization areas) open within the correct workspace.
    • Create/Delete API flows and success dialogs redirect within the current workspace.
    • Added workspace context provider for consistent workspace loading and redirects, including sign-in and “no workspace” guards.
  • Refactor

    • Removed legacy non-workspace routes and updated internal links to workspace-scoped paths.

@linear
Copy link

linear bot commented Aug 11, 2025

@changeset-bot
Copy link

changeset-bot bot commented Aug 11, 2025

⚠️ No Changeset found

Latest commit: e0a4d0f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Aug 11, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
dashboard Ready Ready Preview Comment Sep 11, 2025 0:46am
engineering Ready Ready Preview Comment Sep 11, 2025 0:46am

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Aug 11, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch eng-1893-a-user-can-access-apis-within-workspace-context

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@CLAassistant
Copy link

CLAassistant commented Aug 12, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 13, 2025

Thank you for following the naming conventions for pull request titles! 🙏

@perkinsjr perkinsjr merged commit d3b74c9 into slug-routing Sep 11, 2025
9 checks passed
@perkinsjr perkinsjr deleted the eng-1893-a-user-can-access-apis-within-workspace-context branch September 11, 2025 13:38
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.

4 participants