fix(site): History timeline reads as growth, not repeated commit names - #88
Merged
Merged
Conversation
The dump-history list led with each commit's title, but almost every dump commit
is the same boilerplate ("chore(site): refresh public dump for X import"), so the
list read as a wall of near-identical names. Rework each row to lead with what the
section is actually about — date, record count, and the delta — with the per-category
changes as the detail. shortTitle() strips the conventional-commit prefix and the
dump-refresh boilerplate, keeping only a distinctive tag (e.g. "GSMArena Kaggle") or
nothing. The chart still shows every sync; the list is capped to the recent 8 with a
"N earlier syncs" link to the full commit history.
Refs #1
github-actions
Bot
added
bug
1 task done
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
1 participant
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.
Follow-up to the history fix. The timeline list led with each commit's title, but nearly every dump commit shares the same boilerplate (
chore(site): refresh public dump for X import), so it read as a wall of near-identical names.Now each row leads with the growth story — date · record count · delta — and the per-category change is the detail line.
shortTitle()strips the conventional-commit prefix + dump-refresh boilerplate, keeping only a distinctive tag (GSMArena Kaggle,Phones 2024, …) or nothing for plain refreshes. The chart still shows all 27 syncs; the list is capped to the recent 8 with a "N earlier syncs →" link to the full commit history — so the section stays compact and fits the page.Verified:
npm run buildpasses; new markup ships in the JS bundle.Refs #1
Closes #19