Skip to content

RI-8278 i18n: migrate command markdown texts - #6403

Merged
valkirilov merged 1 commit into
fe/feature/RI-8278/i18n-shared-textsfrom
fe/feature/RI-8278/i18n-command-markdown
Aug 14, 2026
Merged

RI-8278 i18n: migrate command markdown texts#6403
valkirilov merged 1 commit into
fe/feature/RI-8278/i18n-shared-textsfrom
fe/feature/RI-8278/i18n-command-markdown

Conversation

@valkirilov

@valkirilov valkirilov commented Aug 12, 2026

Copy link
Copy Markdown
Member

Summary

Second of 3 PRs for RI-8278. Stacked on #6402 — only the last commit is new here.

  • Workbench: hovering a command suggestion shows a help popup with "Arguments:", "Read more", and labels like "required"/"optional"/"multiple" — these are now translatable.

The English text itself hasn't changed, just how it's delivered.

One thing left as-is: the SQL and JMESPath completion hints in RDI resolve their language once, when they're fetched/built, and don't update if you switch languages afterwards — same limitation either way. Everything else updates live. Not fixed here — small enough to be its own follow-up.

Stack

  1. RI-8278 i18n: migrate shared text and help-text constants #6402
  2. This PR
  3. fe/feature/RI-8278/i18n-keyboard-shortcuts (next)

Test plan

  • Automated checks pass (lint, type-check, translation file checks, existing tests)
  • Manual check in English and Bulgarian: Workbench -> hover a command suggestion

Resolves the last hardcoded English in utils/commands.ts, used to
build the Monaco hover markdown for Redis and SQLite/JMESPath
completions.

References: RI-8278
@valkirilov
valkirilov requested a review from a team as a code owner August 12, 2026 09:49
@valkirilov valkirilov self-assigned this Aug 12, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ 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 1883983. Configure here.

Comment thread redisinsight/ui/src/utils/commands.ts
@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 83.58% 28841/34505
🟡 Branches 69.69% 12257/17589
🟡 Functions 78.65% 7620/9689
🟢 Lines 84.05% 28037/33358

Test suite run success

8032 tests passing in 871 suites.

Report generated by 🧪jest coverage report action from 1883983

@valkirilov
valkirilov merged commit 1c68461 into main Aug 14, 2026
19 checks passed
@valkirilov
valkirilov deleted the fe/feature/RI-8278/i18n-command-markdown branch August 14, 2026 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants