fix(sdk): add icons to PAT details header action menu#1635
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR adds visual icon assets to the PAT details view action menu. Icon imports (Pencil1Icon, Image component, and delete.svg) are added, then wired into the "Update" and "Revoke" menu items as leading icons with no changes to handlers, props, or component logic. ChangesPAT Action Menu Icons
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 35f50b35-aa32-46b3-9ead-55afe2e6232d
📒 Files selected for processing (1)
web/sdk/react/views-new/pat/pat-details-view.tsx
Coverage Report for CI Build 26152199215Coverage remained the same at 42.592%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
Summary
Pencil1Icon, Regenerate usesUpdateIcon, and Revoke uses the shareddelete.svgasset — matching icon usage elsewhere in views-new.