[🕹️] Switching Workspace when api selected leads to 404

[Flo4604]

Preliminary Checks

• I have reviewed the documentation: https://unkey.com/docs

• I have searched for existing issues: https://github.com/unkeyed/unkey/issues

• This issue is not a question, general help request, or anything other than a bug report directly related to Unkey. Please ask questions in our Discord community: https://unkey.com/discord.

Reproduction / Replay Link (Optional)

No response

Issue Summary

When I select an API in a workspace and switch to another workspace, I get a 404 page.

Steps to Reproduce

1. Select a API in a workspace
2. Switch to another workspace
3. See 404 page
   See repro video

Expected behavior

I would like to be redirected to the API overview instead

Other information

No response

Screenshots

switch_workspace.mp4

Version info

-

[linear]

ENG-1380 [🕹️] Switching Workspace when api selected leads to 404

[chronark]

Yes!!

In /apis/[apiId]/page.tsx we should return redirect("/apis") instead of return notFound()

[chronark]

/award 150

[oss-gg]

Awarding Flo4604: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/Flo4604

[AkshayBandi027]

/assign

[oss-gg]

Assigned to @AkshayBandi027! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

[AkshayBandi027]

Hey @chronark !! the problem is occurring across multiple dynamic routes, including API , rate limiting, and authorization. When a user changes their organization, it cannot the specific
data, so its returns notfound()

Two Potential Solutions

Router Push Solution: When the user changes their organization, the router can push to a standard route (e.g., /apis) instead of the current route. in changeOrg function .
router.push("/apis")

Redirect Solution: we need to add redirect in mulitple dynamic route inlcuding api,rate limiting and authorization
. (e.g. In /apis/[apiId]/page.tsx we should return redirect("/apis") instead of return notFound())

[chronark]

We're about to change how the changeOrg stuff works, let's not change that now.
I also think handling this per page and setting a better fallback (ie /apis) is better in general, cause it's more explicit and catches the error even if the root cause is something other than changing orgs

[oss-gg]

@AkshayBandi027, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

[oss-gg]

@AkshayBandi027, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

[mohitejaikumar]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[mohitejaikumar]

@chronark I think no one is working on this can I work on this

[adkibhaskar]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[diwasrimal]

/assign

[oss-gg]

Assigned to @diwasrimal! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

[mohitejaikumar]

@chronark I first asked you to assign it to me 🥹

[chronark]

@mohitejaikumar this month, all issues can be assigned by you via /assign, maintainers don't assign issues manually right now
you can read more on https://oss.gg

[mohitejaikumar]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[no-naame]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[diwasrimal]

/unassign

[oss-gg]

Issue unassigned.

[anuj5926]

/assign

[oss-gg]

Assigned to @anuj5926! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

[oss-gg]

@anuj5926, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

[mohitejaikumar]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[oss-gg]

@anuj5926, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

[Vardhaman619]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[chronark]

cc @mcstepp, for when we change to slug based navigation