-
Notifications
You must be signed in to change notification settings - Fork 619
show deprecation in playground UI #7100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
WalkthroughThe changes introduce a deprecation indicator system for API paths across the UI. A new Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Sidebar
participant MainPage
participant BlueprintData
User->>Sidebar: Render links
Sidebar->>BlueprintData: Get links with `crossedOut` (deprecated) property
Sidebar->>Sidebar: Render link with line-through if crossedOut
User->>MainPage: Navigate to blueprint path
MainPage->>BlueprintData: Fetch path metadata (includes `deprecated`)
MainPage->>MainPage: Render title with line-through and "Deprecated" badge if deprecated
MainPage->>MainPage: Show description for deprecated paths
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (6)
🧰 Additional context used🧬 Code Graph Analysis (1)apps/playground-web/src/app/insight/page.tsx (1)
⏰ Context from checks skipped due to timeout of 90000ms (2)
🔇 Additional comments (12)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7100 +/- ##
=======================================
Coverage 55.81% 55.81%
=======================================
Files 900 900
Lines 57848 57848
Branches 4067 4067
=======================================
Hits 32288 32288
Misses 25454 25454
Partials 106 106
🚀 New features to boost your workflow:
|
size-limit report 📦
|
483ce36 to
7e98519
Compare
171dbe1 to
342aef1
Compare
342aef1 to
b9b3d52
Compare
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
bde14b9 to
438c019
Compare
9fb808d to
cdcdee9
Compare
Merge activity
|
<!-- start pr-codex -->
## PR-Codex overview
This PR focuses on adding a `deprecated` property to various types and components within the application, allowing for better handling and display of deprecated links and features in the UI.
### Detailed summary
- Added `deprecated` property to `pathInfo` in `navLinks.ts`.
- Updated `MinimalBlueprintSpec` type in `utils.ts` to include `deprecated`.
- Modified `SidebarLink` type in `sidebar.tsx` to optionally include `crossedOut`.
- Enhanced UI to visually indicate deprecated items in `page.tsx`.
- Included `deprecated` status for various API endpoints in `insightBlueprints.ts`.
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
cdcdee9 to
20fdd5b
Compare

PR-Codex overview
This PR focuses on adding a
deprecatedproperty to various types and components within the application, allowing for better handling and display of deprecated items in the UI.Detailed summary
deprecatedproperty topathInfoinnavLinks.ts,page.tsx, andinsightBlueprints.ts.MinimalBlueprintSpecinutils.tsto includedeprecated.SidebarIteminsidebar.tsxto conditionally apply a line-through style for deprecated links.page.tsxto show "Deprecated" badge and description for deprecated paths.crossedOutproperty inSidebarLinktype for better visual indication of deprecated links.Summary by CodeRabbit
New Features
Style