RI-8276 i18n: migrate Database Analysis page#6199
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ae6ea8e00d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
valkirilov
force-pushed
the
fe/feature/RI-8276/i18n-analytics-database-analysis
branch
2 times, most recently
from
71b61fd to
e3bd16d
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e3bd16d. Configure here.
Route the Database Analysis page through i18n under analytics.databaseAnalysis.* with Bulgarian translations: page title, header (report select, scan progress, new-report button + tooltip title/body), the Data Summary / Tips tabs, empty states (reports/keys/encrypt — the keys variant links Workbench via <Trans>), recommendations empty state and tutorial, summary-per-data and TTL sections, and the top-namespaces / top-keys tables (headers, tooltips, toggles). The no-namespaces empty state links Tree View via <Trans>. Also migrates the analytics navigation tabs (AnalyticsTabs: Overview / Database Analysis / Slow Log) under analytics.nav.*, and translates the new-report tooltip body at the use site (leaving the shared recommendations constant untouched for the still-English side panels). Extends analytics.units.* with %/B resolved through t(). The TTL tooltip seconds suffix stays literal to match the untranslated duration from the shared truncateNumberToDuration util (same rationale as cluster-details). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
valkirilov
force-pushed
the
fe/feature/RI-8276/i18n-analytics-database-analysis
branch
from
e3bd16d to
f618ca0
Compare
Code Coverage - Frontend unit tests
Test suite run success7763 tests passing in 860 suites. Report generated by 🧪jest coverage report action from 52f7047 |
…8n-analytics-database-analysis # Conflicts: # redisinsight/ui/src/i18n/locales/bg.json # redisinsight/ui/src/i18n/locales/en.json
1 participant
Routes the Database Analysis page through i18next under
analytics.databaseAnalysis.*(+ sharedanalytics.units.*), with Bulgarian translations. Text-only; no behavior change.Note
Low Risk
Presentation-only i18n refactor with no API or analysis logic changes; main risk is missing or mismatched translation keys affecting copy in non-English locales.
Overview
Database Analysis and related Analytics navigation are wired through i18next instead of hard-coded English strings.
User-visible copy on the Database Analysis flow (page title, header actions/progress/tooltips, Data Summary vs Tips tabs, empty states, TTL chart, summary donuts, top keys/namespaces tables, and recommendations) now comes from
analytics.databaseAnalysis.*, with sharedanalytics.units.*for%and byte suffixes. Analytics tab labels useanalytics.nav.*. English and Bulgarian locale entries were added for those keys.Rich messages that embed links (empty “no keys” and “no namespaces”) use
Transwith component placeholders. The analysis header no longer imports analyze tooltip constants from recommendations; cluster vs standalone tooltip text is selected from locale strings instead.Reviewed by Cursor Bugbot for commit 52f7047. Bugbot is set up for automated code reviews on this repo. Configure here.