fix: improve Server Information version and commit SHA display#186
Conversation
- Drop the redundant version chip from the Server Information and Console card toolbars (the version is already listed in the table below). - Show both Lakekeeper and Enterprise versions in Plus/enterprise consoles instead of only one. - Truncate commit SHAs to 7 chars with a hover tooltip showing the full value; the copy button still copies the full SHA. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Walkthrough
ChangesVersion Display Consolidation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
What
Improve the Server Information / Console cards in
ServerOverview:isEnterpriseConsole); OSS shows only the Lakekeeper version.Why
The toolbar chip duplicated the table row. Plus deployments run two versioned artifacts (lakekeeper + enterprise) and need both visible. Full SHAs were too long inline, so they're truncated with the full value one hover/copy away.
BEGIN_COMMIT_OVERRIDE
fix(ui): improve Server Information version and commit SHA display
END_COMMIT_OVERRIDE
🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
UI Improvements