Repository navigation
fix(rankings): correct period labels and descriptions to match rollin… - #7722
CupOfBread wants to merge 1 commit into
Conversation
…g time windows The rankings tabs translated Week/Month/Year as 本周/本月/今年, but the backend computes rolling windows of the last 7/30/365 days, so the labels mislead users about the data range. Rename the tab labels to "Last 7 days", "Last 30 days", and "Last year" with dedicated i18n keys (the existing Week/Month/Year keys stay for other features), and fix the history chart descriptions that still said "past few weeks" / "past month" / "past year".
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
WalkthroughRanking period descriptions and labels now use explicit 7-day, 30-day, and yearly ranges. English, French, Japanese, Russian, Vietnamese, Traditional Chinese, and Chinese locale files add corresponding analytics labels and descriptions. ChangesRanking time ranges
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~8 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The updated ranking labels and chart descriptions match the data windows, and no actionable merge risk remains. Pre-merge checks |
 |
| Check name | Status | Explanation |
|---|---|---|
| Description Check |  |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | |
The title clearly identifies the rankings period-label and description correction, which matches the primary changes in the pull request. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 3… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
A rabbit checks the range labels bright,
Seven days, thirty days, then a year in sight.
Across each locale, the new words appear,
Chart descriptions make the time spans clear.
It hops away, its notes all neat,
And leaves the rankings fresh and sweet.
Comment @coderabbitai help to get the list of available commands.
🔗 关联任务 / Related Issue
🚀 变更类型 / Type of change
📝 变更描述 / Description
排行榜的周期标签「本周 / 本月 / 今年」与后端实际的滚动窗口（最近 7 / 30 / 365 天）不一致，误导用户对数据范围的理解。本次改动：
📸 运行证明 / Proof of Work
bun run i18n:sync通过，所有 locale 无缺失 keybun run build通过✅ 提交前检查项 / Checklist
Summary by CodeRabbit