feat(dashboard): group analytics sections and rename headings#2140
Open
singhanurag0317-bit wants to merge 1 commit into
Open
Conversation
…yanshu-byte-coder#1397 - Top Repositories → Repo Analytics (TopRepos.tsx, dashboard-layout.ts) - PR Breakdown + PR Review Time Trend → PR Analytics — Breakdown/Review Trend (PRBreakdownChart.tsx, PRReviewTrendChart.tsx, dashboard-layout.ts) - PR Metrics → PR Analytics (PRMetrics.tsx, dashboard-layout.ts) - Commits by Time of Day → Activity Overview (CommitTimeChart.tsx, dashboard-layout.ts) - Issue Metrics → Issue Analytics (IssueMetrics.tsx, dashboard-layout.ts) Removes repetitive naming; groups related PR metrics under a unified "PR Analytics" umbrella; aligns widget labels in the customizable dashboard panel. Closes Priyanshu-byte-coder#1397
|
@singhanurag0317-bit is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
Owner
|
This PR now has merge conflicts after recent changes to git fetch origin
git rebase origin/main |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes #1397
Problem
The dashboard had repetitive and overlapping section names:
These naming issues made the dashboard feel cluttered and hard to scan.
Solution
Renamed all affected widget headings and the
DASHBOARD_WIDGET_LABELSregistry to group related metrics under clear analytic umbrella terms, exactly matching the acceptance criteria in #1397:The "PR Analytics" prefix on Breakdown and Review Trend cards makes the grouping relationship immediately obvious without requiring users to rearrange widgets.
Changes
src/lib/dashboard-layout.ts: UpdatedDASHBOARD_WIDGET_LABELSfor 6 widget IDs so the customizable dashboard panel shows updated names in its sidebar/settingssrc/components/PRMetrics.tsx: SectionHeader title"PR Metrics"→"PR Analytics"src/components/PRBreakdownChart.tsx:<h2>text"PR Breakdown"→"PR Analytics — Breakdown"src/components/PRReviewTrendChart.tsx:<h2>text"PR Review Time Trend"→"PR Analytics — Review Trend"src/components/TopRepos.tsx: SectionHeader title →"Repo Analytics"src/components/CommitTimeChart.tsx:<h2>text →"Activity Overview"src/components/IssueMetrics.tsx: any heading/label"Issue Metrics"→"Issue Analytics"Testing
DASHBOARD_WIDGET_LABELS)."PR Breakdown"/"PR Review Time Trend"/"Top Repositories"/"Commit Time"/"Issue Metrics"in the UI — none should appear).