Skip to content

Conversation

@ben-fornefeld
Copy link
Member

@ben-fornefeld ben-fornefeld commented Oct 30, 2025

Note

Replaces the tabbed /settings route with standalone /general and /keys pages and updates redirects, URLs, layout config, and sidebar structure.

  • Routing & URLs:
    • Remove settings parallel routes; add dedicated pages at .../[teamIdOrSlug]/general/page.tsx and .../[teamIdOrSlug]/keys/page.tsx.
    • Update PROTECTED_URLS to include GENERAL and KEYS (remove SETTINGS).
    • Adjust TAB_URL_MAP in src/app/dashboard/route.ts to point settings/team to GENERAL and add keys mapping.
    • Update layout configs to add /dashboard/*/general and /dashboard/*/keys (remove /dashboard/*/settings).
  • Sidebar:
    • Replace Settings link with separate General and API Keys links; reorder Members; update activeMatch patterns.
    • Allow SidebarNavItem.icon to accept custom function components; use SettingsIcon for General.
    • Regroup billing links under billing group and adjust ordering.
  • UI Pages:
    • Implement General (profile, name, info cards) and API Keys (table and create dialog) pages within Frame wrappers.

Written by Cursor Bugbot for commit cef069b. This will update automatically on new commits. Configure here.

- Removed the conditional rendering of the 'Usage' link based on `INCLUDE_BILLING`.
- Renamed the 'Billing' link to 'Usage' and adjusted its properties accordingly.
- Added a new 'Billing' link under the billing group with the correct properties.
- Updated the group categorization for 'Budget' and 'Usage' links to align with billing context.
@ben-fornefeld ben-fornefeld added the improvement Improvements in-code label Oct 30, 2025
@vercel
Copy link

vercel bot commented Oct 30, 2025

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

Project Deployment Preview Comments Updated (UTC)
web Ready Ready Preview Comment Oct 30, 2025 6:38pm
web-juliett Ready Ready Preview Comment Oct 30, 2025 6:38pm

@ben-fornefeld
Copy link
Member Author

bugbot run

cursor[bot]

This comment was marked as outdated.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no bugs!


@ben-fornefeld ben-fornefeld merged commit 2365e18 into main Oct 30, 2025
6 checks passed
@ben-fornefeld ben-fornefeld deleted the improve/sidebar-layout branch October 30, 2025 19:09
ben-fornefeld added a commit that referenced this pull request Nov 3, 2025
<!-- CURSOR_SUMMARY -->
> [!NOTE]
> Replaces the tabbed `/settings` route with standalone `/general` and
`/keys` pages and updates redirects, URLs, layout config, and sidebar
structure.
> 
> - **Routing & URLs**:
> - Remove `settings` parallel routes; add dedicated `pages` at
`.../[teamIdOrSlug]/general/page.tsx` and
`.../[teamIdOrSlug]/keys/page.tsx`.
> - Update `PROTECTED_URLS` to include `GENERAL` and `KEYS` (remove
`SETTINGS`).
> - Adjust `TAB_URL_MAP` in `src/app/dashboard/route.ts` to point
`settings/team` to `GENERAL` and add `keys` mapping.
> - Update layout configs to add `/dashboard/*/general` and
`/dashboard/*/keys` (remove `/dashboard/*/settings`).
> - **Sidebar**:
> - Replace `Settings` link with separate `General` and `API Keys`
links; reorder `Members`; update `activeMatch` patterns.
> - Allow `SidebarNavItem.icon` to accept custom function components;
use `SettingsIcon` for General.
>   - Regroup billing links under `billing` group and adjust ordering.
> - **UI Pages**:
> - Implement `General` (profile, name, info cards) and `API Keys`
(table and create dialog) pages within `Frame` wrappers.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
cef069b. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvements in-code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants