fix(sdk): PAT page UI fixes and copy updates by rohilsurana · Pull Request #1646 · raystack/frontier

rohilsurana · 2026-05-25T07:29:03Z

Summary

Remove href="#" from breadcrumb items across all detail views (PAT, projects, service accounts, teams)
Change PAT warning callout from red alert to yellow attention type with ExclamationTriangleIcon
Fix project chips baseline alignment with label text in PAT detail view
Update PAT page header and empty state copy
Show -- for empty "Last used" field in PAT table
Remove unused MouseEvent import from service account details view
Add PAT list and detail routes to OrganizationProfile router with sidebar navigation entry

Test plan

…arning callout

vercel · 2026-05-25T07:29:08Z

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

Project	Deployment	Actions	Updated (UTC)
frontier	Ready	Preview, Comment	May 25, 2026 9:30am

coderabbitai · 2026-05-25T07:29:18Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 333d9a0c-b9da-44d1-af06-69508305ef98

📥 Commits

Reviewing files that changed from the base of the PR and between a50ba67 and b4091c6.

📒 Files selected for processing (5)

web/sdk/react/components/organization/pat/index.tsx
web/sdk/react/components/organization/pat/pat.tsx
web/sdk/react/components/organization/routes.tsx
web/sdk/react/components/organization/sidebar/helpers.ts
web/sdk/react/views-new/pat/components/pat-columns.tsx

📝 Walkthrough

Summary by CodeRabbit

New Features
- Added Personal Access Tokens management with dedicated pages and sidebar navigation.
- Implemented PAT details view with navigation controls.
Bug Fixes
- Improved display of missing usage data in PAT listings.
Style
- Refined warning dialog styling and visual indicators.
- Enhanced breadcrumb navigation interactions across detail pages.
Documentation
- Updated user-facing descriptions for clarity on PAT security and access purposes.

Walkthrough

Breadcrumb click handlers refactored to use direct onClick callbacks across detail views. PAT created dialog callout/icon and PAT detail-row alignment updated. PAT view copy and last-used column adjusted. Workspace PAT routes, pages, and sidebar entry added.

Changes

UI Interaction and Styling Updates

Layer / File(s)	Summary
Breadcrumb Navigation Pattern Refactoring `web/sdk/react/views-new/pat/pat-details-view.tsx`, `web/sdk/react/views-new/projects/project-details-view.tsx`, `web/sdk/react/views-new/service-accounts/service-account-details-view.tsx`, `web/sdk/react/views-new/teams/team-details-view.tsx`	Breadcrumb items in PAT, Project, Service Account, and Team detail views updated from `href="#"` anchors with `preventDefault()` to direct `onClick` handlers using optional chaining (`onNavigateTo...?.()`). Service accounts view import updated to remove unused `MouseEvent` type.
PAT Dialog and Detail View Styling `web/sdk/react/views-new/pat/components/pat-created-dialog.tsx`, `web/sdk/react/views-new/pat/pat-details-view.module.css`	PAT created dialog warning callout updated from `type="alert"` to `type="attention"` and now uses `ExclamationTriangleIcon` instead of `InfoCircledIcon`. PAT detail row CSS `align-items` changed from `flex-start` to `baseline`.
PAT View Text and List Column `web/sdk/react/views-new/pat/pat-view.tsx`, `web/sdk/react/views-new/pat/components/pat-columns.tsx`	View header description and empty-state subheading updated to emphasize managing personal access tokens for securely accessing app resources and APIs. `Last used` column now shows `--` when `pat.usedAt` is missing.
Workspace PAT Routes and Sidebar `web/sdk/react/components/organization/pat/index.tsx`, `web/sdk/react/components/organization/pat/pat.tsx`, `web/sdk/react/components/organization/routes.tsx`, `web/sdk/react/components/organization/sidebar/helpers.ts`	Adds `WorkspacePats` client component and `PatPage` routed page, registers `/pats` and `/pats/$patId` in organization routes, and adds a "Personal Access Tokens" sidebar nav entry pointing to `/pats`.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

raystack/frontier#1533: Earlier PAT list/details feature work that introduced the PAT components modified here.

Suggested reviewers

rsbh
paanSinghCoder
rohanchkrabrty

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

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.}

coveralls · 2026-05-25T07:35:14Z

Coverage Report for CI Build 26393564497

Coverage remained the same at 42.847%

Details

Coverage remained the same as the base build.
Patch coverage: No coverable lines changed in this PR.
No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.

Coverage Stats


Relevant Lines:	37891
Covered Lines:	16235
Line Coverage:	42.85%
Coverage Strength:	11.96 hits per line

💛 - Coveralls

coderabbitai

Actionable comments posted: 1

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 32e7df8b-57a0-43d5-be5e-cf52de3e5204

📥 Commits

Reviewing files that changed from the base of the PR and between 9e2e75a and 95e0da1.

📒 Files selected for processing (6)

web/sdk/react/views-new/pat/components/pat-created-dialog.tsx
web/sdk/react/views-new/pat/components/pat-form-dialog.tsx
web/sdk/react/views-new/pat/pat-details-view.tsx
web/sdk/react/views-new/projects/project-details-view.tsx
web/sdk/react/views-new/service-accounts/service-account-details-view.tsx
web/sdk/react/views-new/teams/team-details-view.tsx

…ionProfile

fix(sdk): fix disabled radio cursor, breadcrumb navigation, and PAT w…

64179cb

…arning callout

vercel Bot deployed to Preview May 25, 2026 07:29 View deployment

fix(sdk): remove unused MouseEvent import

95e0da1

vercel Bot deployed to Preview May 25, 2026 07:31 View deployment

fix(sdk): align project chips with label text using baseline alignment

1bddec4

vercel Bot deployed to Preview May 25, 2026 07:37 View deployment

coderabbitai Bot reviewed May 25, 2026

View reviewed changes

Comment thread web/sdk/react/views-new/pat/pat-details-view.tsx Outdated

fix(sdk): update PAT page copy for header and empty state

b2e6260

vercel Bot deployed to Preview May 25, 2026 07:40 View deployment

rohilsurana changed the title ~~fix(sdk): fix disabled radio cursor, breadcrumb navigation, and PAT warning callout~~ fix(sdk): PAT page UI fixes and copy updates May 25, 2026

revert(sdk): remove disabled radio cursor override

b3f7f8c

vercel Bot deployed to Preview May 25, 2026 07:52 View deployment

fix(sdk): remove unnecessary cursor pointer style from breadcrumb items

a50ba67

vercel Bot deployed to Preview May 25, 2026 08:04 View deployment

fix(sdk): show -- for empty last used field in PAT table

bcc0ba3

vercel Bot deployed to Preview May 25, 2026 08:13 View deployment

feat(sdk): add PAT routes and sidebar entry to OrganizationProfile

b4091c6

vercel Bot deployed to Preview May 25, 2026 08:41 View deployment

rsbh approved these changes May 25, 2026

View reviewed changes

refactor(sdk): remove old PAT routes and sidebar entry from Organizat…

86e6525

…ionProfile

vercel Bot deployed to Preview May 25, 2026 09:30 View deployment

AmanGIT07 merged commit e24bcb7 into main May 25, 2026
8 checks passed

AmanGIT07 deleted the pat-sdk-fixes branch May 25, 2026 11:07

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(sdk): PAT page UI fixes and copy updates#1646

fix(sdk): PAT page UI fixes and copy updates#1646
AmanGIT07 merged 9 commits into
mainfrom
pat-sdk-fixes

rohilsurana commented May 25, 2026 •

edited

Loading

Uh oh!

vercel Bot commented May 25, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented May 25, 2026 •

edited

Loading

Summary by CodeRabbit

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

coveralls commented May 25, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

rohilsurana commented May 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Uh oh!

vercel Bot commented May 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented May 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

coveralls commented May 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report for CI Build 26393564497

Coverage remained the same at 42.847%

Details

Uncovered Changes

Coverage Regressions

Coverage Stats

💛 - Coveralls

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

rohilsurana commented May 25, 2026 •

edited

Loading

vercel Bot commented May 25, 2026 •

edited

Loading

coderabbitai Bot commented May 25, 2026 •

edited

Loading

coveralls commented May 25, 2026 •

edited

Loading