chore(i18n): polish en_US wording and terminology across all pages - #2240
chore(i18n): polish en_US wording and terminology across all pages#2240710leo wants to merge 2 commits into
Conversation
- Unify industry terms: Blackout/Shield -> Mute/Muting rules, Execution duration/interval -> For duration/Evaluation interval, alarm -> alert, machine -> host, curve -> series, Datasource -> Data source - Fix mistranslations against zh_CN (host clock offset, subscription for_duration, inhibit scenario, event_recover processor = Self-healing) - Fix typos and grammar: binded/unbinded, submited, persent, buisness, Templete, updator, "Are you sure to", full-width punctuation - Normalize UI labels to sentence case across pages and plugins
|
Warning Review limit reached
Next review available in: 34 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThe PR updates English locale strings across shared components, application pages, alerting and workflow interfaces, dashboards, explorers, and data-source plugins. Changes standardize sentence case, terminology, validation text, labels, and explanatory guidance without changing exported entities. ChangesEnglish localization updates
Estimated code review effort: 2 (Simple) | ~15 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Note
Due to the large number of review comments, Critical, Major severity comments were prioritized as inline comments.
🟡 Minor comments (24)
src/plugins/clickHouse/locale/en_US.ts-55-57 (1)
55-57: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winComplete the sentence-case normalization for related labels.
The PR changes visible labels to sentence case but leaves neighboring references or actions in title case. Apply these updates:
src/plugins/clickHouse/locale/en_US.ts#L55-L57: changeTime Series modetoTime series mode.src/plugins/doris/locale/en_US.ts#L79-L80: changeTime Series modetoTime series mode.src/plugins/elasticsearch/locale/en_US.ts#L42-L43: change the siblingadd_to.btnvalue fromAdd TotoAdd to.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/plugins/clickHouse/locale/en_US.ts` around lines 55 - 57, Normalize the related labels to sentence case: in src/plugins/clickHouse/locale/en_US.ts lines 55-57 and src/plugins/doris/locale/en_US.ts lines 79-80, change “Time Series mode” to “Time series mode”; in src/plugins/elasticsearch/locale/en_US.ts lines 42-43, update the sibling add_to.btn value from “Add To” to “Add to”.src/pages/alertRules/locale/en_US.ts-121-124 (1)
121-124: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winSpecify the host clock offset.
host.trigger.key.offsetis a host-specific trigger.Time offsetdoes not identify the affected clock. Change the label toHost clock offset.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/alertRules/locale/en_US.ts` around lines 121 - 124, The offset property label in the host trigger keys object does not specify that the time offset is host-specific. Update the offset property value from 'Time offset' to 'Host clock offset' to clarify that this is a host-specific trigger affecting the host's clock.src/pages/alertRules/locale/en_US.ts-130-140 (1)
130-140: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winCorrect the duplicated duration guidance.
Usually the for durationandDuring the for durationare ungrammatical. UseThe configured durationandDuring this durationin both strings.Proposed fix
- 'Usually the for duration is longer than the evaluation interval. During the for duration, the query is evaluated multiple times at the evaluation interval, and an alert fires only if the condition is met every time. If set to 0, an alert fires as soon as the condition is met once.' + 'The configured duration is usually longer than the evaluation interval. During this duration, the query is evaluated multiple times at the evaluation interval, and an alert fires only if the condition is met every time. If set to 0, an alert fires as soon as the condition is met once.'🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/alertRules/locale/en_US.ts` around lines 130 - 140, Update both prom_for_duration_tip strings under the alert rule query and metric query sections: replace “Usually the for duration” with “The configured duration” and “During the for duration” with “During this duration,” preserving the remaining guidance.src/pages/warning/subscribe/locale/en_US.ts-47-47 (1)
47-47: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winInclude the unit in the duration example.
The field is measured in seconds, but the tip says “with 300 configured.” Use “with 300 seconds configured” to remove the ambiguity.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/warning/subscribe/locale/en_US.ts` at line 47, Update the duration example in the warning subscription locale text to say “with 300 seconds configured” instead of “with 300 configured,” preserving the rest of the explanatory text.src/pages/notificationChannels/locale/en_US.ts-65-65 (1)
65-65: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winUse “Skip TLS verification” for the TLS setting.
Skip TLS verifyis not a standard noun phrase. State the operation clearly because this label describes a security-sensitive option.Proposed wording
- insecure_skip_verify: 'Skip TLS verify', + insecure_skip_verify: 'Skip TLS verification',Also applies to: 86-87
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/notificationChannels/locale/en_US.ts` at line 65, Update the insecure_skip_verify locale label to use the clear wording “Skip TLS verification,” and apply the same wording to the additional occurrence referenced by the review.src/pages/notificationChannels/locale/en_US.ts-63-63 (1)
63-63: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winUse “Number of retries” for
retry_times.
Retry timesis not idiomatic and can be confused with a time interval. Use one label for HTTP, FlashDuty, and PagerDuty.Proposed wording
- retry_times: 'Retry times', + retry_times: 'Number of retries',Also applies to: 110-110, 119-119
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/notificationChannels/locale/en_US.ts` at line 63, Update the retry_times labels in the HTTP, FlashDuty, and PagerDuty notification channel translations to use “Number of retries” instead of “Retry times,” keeping the same translation key and consistent wording across all three entries.src/pages/warning/subscribe/locale/en_US.ts-55-55 (1)
55-55: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winFix the grammar in the tag and group validation message.
Fill in at least one of tags or receiving groupsis incomplete. UseFill in at least one tag or receiving group.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/warning/subscribe/locale/en_US.ts` at line 55, Update the tags_groups_require validation message to read “Fill in at least one tag or receiving group,” replacing the grammatically incomplete wording while preserving the existing localization key.src/pages/warning/shield/locale/en_US.ts-26-26 (1)
26-26: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winMake the broad-scope warning precise.
Line [26] is ambiguous about whether one or both filters are absent. It also says “all alert events,” while the same locale limits muting by severity and time. State that the rule can mute all matching events in the selected business group.
Proposed wording
- scope_unlimited_tip: 'No data source or event tag is configured; this rule will mute all alert events of the selected business group. Please confirm', + scope_unlimited_tip: 'Neither a data source nor an event-tag filter is configured; this rule can mute all alert events in the selected business group that match the configured severity and time conditions. Please confirm',🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/warning/shield/locale/en_US.ts` at line 26, Update the scope_unlimited_tip locale text to explicitly cover the case where either or both filters are unconfigured, and describe the impact as muting all matching events in the selected business group while preserving the existing severity and time constraints.src/pages/notificationRules/locale/en_US.ts-194-194 (1)
194-194: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winUse a natural phrase for aggregation intervals.
too large aggregation time intervalsis not idiomatic and makes the warning harder to read. Useoverly long aggregation intervals.Proposed wording
- default_duration_tip: 'Please note that too large aggregation time intervals will cause alert delivery delays', + default_duration_tip: 'Please note that overly long aggregation intervals delay alert delivery',🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/notificationRules/locale/en_US.ts` at line 194, Update the default_duration_tip translation to replace “too large aggregation time intervals” with the natural phrase “overly long aggregation intervals,” preserving the rest of the warning.src/pages/notificationTemplates/locale/en_US.ts-18-18 (1)
18-18: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winClarify the media-type field reference.
Fields that can be used in the media type, referenced by ...is grammatically incomplete. Preserve{{contentKey}}and state the reference format directly.Proposed wording
- tip: 'Fields that can be used in the media type, referenced by $tpl.{{contentKey}}', + tip: 'Fields available in the media type are referenced as $tpl.{{contentKey}}',🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/notificationTemplates/locale/en_US.ts` at line 18, Update the tip value in the notification template locale to clearly state the media-type field reference format, preserving the {{contentKey}} placeholder and correcting the incomplete grammar.src/pages/warning/shield/locale/en_US.ts-3-3 (1)
3-3: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winMake the edit error message a complete sentence.
Missing required parameters, unable to edit, please contact the administratoris a comma splice. State the failure and the next action directly.Proposed wording
- edit_missing_params: 'Missing required parameters, unable to edit, please contact the administrator', + edit_missing_params: 'Required parameters are missing, so this rule cannot be edited. Please contact an administrator',🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/warning/shield/locale/en_US.ts` at line 3, Update the edit_missing_params locale string to use complete sentences rather than joining independent clauses with a comma, while clearly stating the edit failure and directing the user to contact the administrator.src/pages/siteSettings/locale/en_US.ts-5-5 (1)
5-5: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winMake the image-size hints grammatical.
Lines 5, 7, 9, and 11 use
Enter the image URL, recommended size .... The comma joins two separate instructions. Use two sentences consistently.Proposed wording
- favicon_url_tip: 'Enter the image URL, recommended size 32x32', + favicon_url_tip: 'Enter the image URL. Recommended size: 32x32.', - login_page_logo_url_tip: 'Enter the image URL, recommended size 120x38', + login_page_logo_url_tip: 'Enter the image URL. Recommended size: 120x38.', - menu_big_logo_url_tip: 'Enter the image URL, recommended size 120x38', + menu_big_logo_url_tip: 'Enter the image URL. Recommended size: 120x38.', - menu_small_logo_url_tip: 'Enter the image URL, recommended size 38x38', + menu_small_logo_url_tip: 'Enter the image URL. Recommended size: 38x38.',Also applies to: 7-7, 9-9, 11-11
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/siteSettings/locale/en_US.ts` at line 5, Update the image-size hints for favicon_url_tip and the corresponding entries on lines 7, 9, and 11 to use two grammatical sentences instead of joining the instructions with a comma, preserving the existing URLs and size recommendations.src/pages/dashboard/locale/en_US.ts-61-61 (1)
61-61: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winSeparate the feedback link from the import restriction.
Line 61 ends with
can be importedand immediately appends<a>feedback</a>. The rendered guidance is an incomplete phrase. Add punctuation and an explicit action for the link while preserving the<a>placeholder.Proposed wording
- import_grafana_tip: 'Only dashboards using Prometheus data sources, and the chart types and features supported by Nightingale, can be imported <a>feedback</a>', + import_grafana_tip: 'Only dashboards that use Prometheus data sources and chart types and features supported by Nightingale can be imported. <a>Send feedback</a>',🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/dashboard/locale/en_US.ts` at line 61, Update the import_grafana_tip translation so the import restriction ends with proper punctuation, then add a separate explicit feedback action while preserving the existing <a> placeholder for the link.src/pages/targets/locale/en_US.ts-28-32 (1)
28-32: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winMake the heartbeat ranges non-overlapping.
Lines 29 and 30 both describe a heartbeat “within” a limit. The yellow range overlaps the green range, so the explanation does not identify the middle interval. State the interval explicitly.
Proposed wording
- Heartbeat within 3 minutes: yellow <1 /> - No heartbeat within 3 minutes: red + Heartbeat between 1 and 3 minutes: Yellow <1 /> + No heartbeat for more than 3 minutes: Red🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/targets/locale/en_US.ts` around lines 28 - 32, Update the update_at_tip translation so the yellow heartbeat range explicitly covers the interval after 1 minute through 3 minutes, while preserving the green “within 1 minute” and red “no heartbeat within 3 minutes” descriptions.src/pages/help/SSOConfigs/locale/en_US.ts-21-21 (1)
21-21: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winRewrite the
corpId_tiptext as a complete sentence.
Corporation ID, you can find the CorpId ...is a comma splice. Use a colon or two sentences, for example:Corporation ID: You can find the CorpId on the homepage of the DingTalk Open Platform.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/help/SSOConfigs/locale/en_US.ts` at line 21, Update the corpId_tip translation to use grammatically complete sentence structure by replacing the comma splice with a colon or separate sentences, while preserving the existing meaning and guidance.src/pages/logExplorer/locale/en_US.ts-16-16 (1)
16-16: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winUse the existing
Raw logsmode label.
mode.raw_logsisRaw logs, butconfirm_contentsaysRaw Logs mode. UseRaw logs modeto keep one label and sentence case.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/logExplorer/locale/en_US.ts` at line 16, Update the confirm_content string to use the existing mode.raw_logs label wording, changing “Raw Logs mode” to “Raw logs mode” while preserving the rest of the message.src/pages/help/SSOConfigs/locale/en_US.ts-23-23 (1)
23-23: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winUse sentence case for
client_secret.Change
Client SecrettoClient secret. KeepClient IDbecauseIDis an acronym.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/help/SSOConfigs/locale/en_US.ts` at line 23, Update the client_secret label in the SSOConfigs locale to use sentence case as “Client secret”; leave the Client ID label unchanged because ID is an acronym.src/pages/logExplorer/locale/en_US.ts-64-64 (1)
64-64: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winUse the plural form for the data-source type title.
The key and following description refer to multiple data source types, but the title says
No available data source type. Change it toNo available data source types.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/logExplorer/locale/en_US.ts` at line 64, Update the no_supported_datasource_types_title translation value in the locale resource to use the plural wording “No available data source types,” leaving the key and surrounding translations unchanged.src/pages/explorer/locale/en_US.ts-63-63 (1)
63-63: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winName the download action clearly.
Query logs first, then click downloadis ambiguous. UseQuery logs first, then click the download buttonor the exact button label.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/explorer/locale/en_US.ts` at line 63, Update the emptyText translation in en_US so the download action is explicit, replacing “click download” with “click the download button” or the exact visible button label while preserving the rest of the message.src/pages/explorer/locale/en_US.ts-70-70 (1)
70-70: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winMake the field-existence action label grammatical.
Filter for field presentis not a complete noun phrase. UseFilter documents with this fieldto match the equivalent log explorer action.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/explorer/locale/en_US.ts` at line 70, Update the exists action label in the locale translation to “Filter documents with this field,” matching the equivalent log explorer wording and preserving the existing translation key.src/pages/logExplorer/locale/en_US.ts-31-31 (1)
31-31: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winComplete the field-statistics message.
Field statistics not enabled, unable to perform statistical analysisomitsareand joins two clauses with a comma. UseField statistics are not enabled, so statistical analysis is not available.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/logExplorer/locale/en_US.ts` at line 31, Update the unindexable locale message in the log explorer translations to exactly state that field statistics are not enabled and statistical analysis is not available, using “are” and a “so” clause instead of the current comma-separated wording.src/pages/explorer/locale/en_US.ts-20-20 (1)
20-20: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winComplete both field-presence messages.
Both strings omit
is not presentin the second clause. The explorer message also omitsthebeforemapping.
src/pages/explorer/locale/en_US.ts#L20-L20: useThe field exists in the mapping but is not present in the 500 sampled documents.src/pages/logExplorer/locale/en_US.ts#L58-L58: useThis field exists in the mapping but is not present in the 500 displayed documents.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/explorer/locale/en_US.ts` at line 20, Update fieldValues_topnNoData in src/pages/explorer/locale/en_US.ts at lines 20-20 to use “The field exists in the mapping but is not present in the 500 sampled documents”. Update the corresponding field-presence message in src/pages/logExplorer/locale/en_US.ts at lines 58-58 to use “This field exists in the mapping but is not present in the 500 displayed documents”.src/pages/help/migrate/locale/en_US.ts-12-12 (1)
12-12: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winUse
Migratefor the button label.
migrateis rendered on both the primary “migrate” page button and the modal submit button, while the help text says “Click the migrate button”. KeepMigrationonly for section/title copy such asmodal:title.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/help/migrate/locale/en_US.ts` at line 12, Update the `migrate` locale string to `Migrate` for the primary and modal submit button labels, while preserving `Migration` for section and title text such as `modal:title`.src/pages/log/IndexPatterns/locale/en_US.ts-92-92 (1)
92-92: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winFix the regular-expression example.
The current example displays
:(d+)$, which matches the letterdand cannot extract1740920581from the sample value. Escape the backslash in the TypeScript string so the UI displays:(\d+)$.Proposed fix
- 'log-reg-extract': "e.g. for address field 17:356:256291904:1740920581, extracting with ':(d+)$' generates a new field phone = 1740920581", + 'log-reg-extract': "e.g. for address field 17:356:256291904:1740920581, extracting with ':(\\d+)$' generates a new field phone = 1740920581",🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/log/IndexPatterns/locale/en_US.ts` at line 92, Update the `log-reg-extract` locale string to escape the backslash in the regular-expression example, so the TypeScript source renders `:(\d+)$` in the UI and correctly represents matching the trailing digits.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Minor comments:
In `@src/pages/alertRules/locale/en_US.ts`:
- Around line 121-124: The offset property label in the host trigger keys object
does not specify that the time offset is host-specific. Update the offset
property value from 'Time offset' to 'Host clock offset' to clarify that this is
a host-specific trigger affecting the host's clock.
- Around line 130-140: Update both prom_for_duration_tip strings under the alert
rule query and metric query sections: replace “Usually the for duration” with
“The configured duration” and “During the for duration” with “During this
duration,” preserving the remaining guidance.
In `@src/pages/dashboard/locale/en_US.ts`:
- Line 61: Update the import_grafana_tip translation so the import restriction
ends with proper punctuation, then add a separate explicit feedback action while
preserving the existing <a> placeholder for the link.
In `@src/pages/explorer/locale/en_US.ts`:
- Line 63: Update the emptyText translation in en_US so the download action is
explicit, replacing “click download” with “click the download button” or the
exact visible button label while preserving the rest of the message.
- Line 70: Update the exists action label in the locale translation to “Filter
documents with this field,” matching the equivalent log explorer wording and
preserving the existing translation key.
- Line 20: Update fieldValues_topnNoData in src/pages/explorer/locale/en_US.ts
at lines 20-20 to use “The field exists in the mapping but is not present in the
500 sampled documents”. Update the corresponding field-presence message in
src/pages/logExplorer/locale/en_US.ts at lines 58-58 to use “This field exists
in the mapping but is not present in the 500 displayed documents”.
In `@src/pages/help/migrate/locale/en_US.ts`:
- Line 12: Update the `migrate` locale string to `Migrate` for the primary and
modal submit button labels, while preserving `Migration` for section and title
text such as `modal:title`.
In `@src/pages/help/SSOConfigs/locale/en_US.ts`:
- Line 21: Update the corpId_tip translation to use grammatically complete
sentence structure by replacing the comma splice with a colon or separate
sentences, while preserving the existing meaning and guidance.
- Line 23: Update the client_secret label in the SSOConfigs locale to use
sentence case as “Client secret”; leave the Client ID label unchanged because ID
is an acronym.
In `@src/pages/log/IndexPatterns/locale/en_US.ts`:
- Line 92: Update the `log-reg-extract` locale string to escape the backslash in
the regular-expression example, so the TypeScript source renders `:(\d+)$` in
the UI and correctly represents matching the trailing digits.
In `@src/pages/logExplorer/locale/en_US.ts`:
- Line 16: Update the confirm_content string to use the existing mode.raw_logs
label wording, changing “Raw Logs mode” to “Raw logs mode” while preserving the
rest of the message.
- Line 64: Update the no_supported_datasource_types_title translation value in
the locale resource to use the plural wording “No available data source types,”
leaving the key and surrounding translations unchanged.
- Line 31: Update the unindexable locale message in the log explorer
translations to exactly state that field statistics are not enabled and
statistical analysis is not available, using “are” and a “so” clause instead of
the current comma-separated wording.
In `@src/pages/notificationChannels/locale/en_US.ts`:
- Line 65: Update the insecure_skip_verify locale label to use the clear wording
“Skip TLS verification,” and apply the same wording to the additional occurrence
referenced by the review.
- Line 63: Update the retry_times labels in the HTTP, FlashDuty, and PagerDuty
notification channel translations to use “Number of retries” instead of “Retry
times,” keeping the same translation key and consistent wording across all three
entries.
In `@src/pages/notificationRules/locale/en_US.ts`:
- Line 194: Update the default_duration_tip translation to replace “too large
aggregation time intervals” with the natural phrase “overly long aggregation
intervals,” preserving the rest of the warning.
In `@src/pages/notificationTemplates/locale/en_US.ts`:
- Line 18: Update the tip value in the notification template locale to clearly
state the media-type field reference format, preserving the {{contentKey}}
placeholder and correcting the incomplete grammar.
In `@src/pages/siteSettings/locale/en_US.ts`:
- Line 5: Update the image-size hints for favicon_url_tip and the corresponding
entries on lines 7, 9, and 11 to use two grammatical sentences instead of
joining the instructions with a comma, preserving the existing URLs and size
recommendations.
In `@src/pages/targets/locale/en_US.ts`:
- Around line 28-32: Update the update_at_tip translation so the yellow
heartbeat range explicitly covers the interval after 1 minute through 3 minutes,
while preserving the green “within 1 minute” and red “no heartbeat within 3
minutes” descriptions.
In `@src/pages/warning/shield/locale/en_US.ts`:
- Line 26: Update the scope_unlimited_tip locale text to explicitly cover the
case where either or both filters are unconfigured, and describe the impact as
muting all matching events in the selected business group while preserving the
existing severity and time constraints.
- Line 3: Update the edit_missing_params locale string to use complete sentences
rather than joining independent clauses with a comma, while clearly stating the
edit failure and directing the user to contact the administrator.
In `@src/pages/warning/subscribe/locale/en_US.ts`:
- Line 47: Update the duration example in the warning subscription locale text
to say “with 300 seconds configured” instead of “with 300 configured,”
preserving the rest of the explanatory text.
- Line 55: Update the tags_groups_require validation message to read “Fill in at
least one tag or receiving group,” replacing the grammatically incomplete
wording while preserving the existing localization key.
In `@src/plugins/clickHouse/locale/en_US.ts`:
- Around line 55-57: Normalize the related labels to sentence case: in
src/plugins/clickHouse/locale/en_US.ts lines 55-57 and
src/plugins/doris/locale/en_US.ts lines 79-80, change “Time Series mode” to
“Time series mode”; in src/plugins/elasticsearch/locale/en_US.ts lines 42-43,
update the sibling add_to.btn value from “Add To” to “Add to”.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 2fcd0080-2197-45c0-88cc-a843c399ac1c
📒 Files selected for processing (63)
src/components/AiChatNG/locale/en_US.tssrc/components/BlankBusinessPlaceholder/locale/en_US.tssrc/components/CronPattern/locale/en_US.tssrc/components/DatasourceSelect/locale/en_US.tssrc/components/InputEnlarge/locale/en_US.tssrc/components/KQLInput/locale/en_US.tssrc/components/KVTagSelect/locale/en_US.tssrc/components/LogQL/locale/en_US.tssrc/components/PromGraphCpt/locale/en_US.tssrc/components/PromQueryBuilder/locale/en_US.tssrc/components/SideMenu/locale/en_US.tssrc/components/TableColumnSelect/locale/en_US.tssrc/components/TimeRangePicker/locale/en_US.tssrc/components/ViewSelect/locale/en_US.tssrc/components/menu/locale/en_US.tssrc/components/pageLayout/locale/en_US.tssrc/locales/common/locale/en_US.tssrc/locales/datasource/locale/en_US.tssrc/pages/account/locale/en_US.tssrc/pages/aiConfig/agents/locale/en_US.tssrc/pages/aiConfig/llmConfigs/locale/en_US.tssrc/pages/aiConfig/skills/locale/en_US.tssrc/pages/alertCurEvent/locale/en_US.tssrc/pages/alertRules/locale/en_US.tssrc/pages/builtInComponents/locale/en_US.tssrc/pages/dashboard/Components/UnitPicker/locale/en_US.tssrc/pages/dashboard/locale/en_US.tssrc/pages/datasource/locale/en_US.tssrc/pages/embeddedProduct/locale/en_US.tssrc/pages/event/locale/en_US.tssrc/pages/eventPipeline/locale/en_US.tssrc/pages/explorer/locale/en_US.tssrc/pages/help/NotificationSettings/locale/en_US.tssrc/pages/help/SSOConfigs/locale/en_US.tssrc/pages/help/migrate/locale/en_US.tssrc/pages/help/servers/locale/en_US.tssrc/pages/historyEvents/locale/en_US.tssrc/pages/hosts/locale/en_US.tssrc/pages/landing/locale/en_US.tssrc/pages/log/IndexPatterns/locale/en_US.tssrc/pages/logExplorer/locale/en_US.tssrc/pages/metricsBuiltin/locale/en_US.tssrc/pages/monitor/object/locale/en_US.tssrc/pages/notFound/locale/en_US.tssrc/pages/notificationChannels/locale/en_US.tssrc/pages/notificationRules/locale/en_US.tssrc/pages/notificationTemplates/locale/en_US.tssrc/pages/permissions/locale/en_US.tssrc/pages/recordingRules/locale/en_US.tssrc/pages/siteSettings/locale/en_US.tssrc/pages/targets/locale/en_US.tssrc/pages/traceCpt/locale/en_US.tssrc/pages/warning/shield/locale/en_US.tssrc/pages/warning/subscribe/locale/en_US.tssrc/plugins/TDengine/locale/en_US.tssrc/plugins/clickHouse/locale/en_US.tssrc/plugins/doris/locale/en_US.tssrc/plugins/elasticsearch/locale/en_US.tssrc/plugins/iotdb/locale/en_US.tssrc/plugins/loki/locale/en_US.tssrc/plugins/mysql/locale/en_US.tssrc/plugins/pgsql/locale/en_US.tssrc/plugins/victorialogs/locale/en_US.ts
…nglish in UI strings
- user: 'Selected {num} items' used a single brace so i18next never
interpolated it; English users saw the literal placeholder. Other
languages already used {{num}}
- user: translate perm_flag_0/1 (were left as the raw 'ro' / 'rw'),
reword the two empty-state fragments and the business group note
- EnhancedTable: action column fell back to a hardcoded Chinese title
- RelativeTimeRangePicker: default placeholder was hardcoded Chinese
- AiChatNG: fix user-visible English prompts ('busi group', 'Host
machine', missing question marks, machine -> host)
f90a184 to
cc065c8
Compare
Summary by CodeRabbit