Skip to content

i18n(ja): fix straddling-bracket defects around New Relic's Import dashboard button - #23540

Merged
qiancai merged 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:fix-new-relic-import-dashboard-bracket-2026-08-17
Aug 17, 2026
Merged

i18n(ja): fix straddling-bracket defects around New Relic's Import dashboard button#23540
qiancai merged 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:fix-new-relic-import-dashboard-bracket-2026-08-17

Conversation

@yahonda

@yahonda yahonda commented Aug 17, 2026

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

monitor-new-relic-integration.md had two pre-existing MT defects on the "Import dashboard" step, found during review of #23535 (unrelated to that PR — this text was never touched since it wasn't confirmed as literal English UI text, being a third-party New Relic console step):

  • 「ダッシュ**ボードのインポート」**をクリックします — the word ダッシュボード was split in half across a bold marker boundary.
  • **[ダッシュボードをインポート] を**クリックします — decoration brackets and the を particle were trapped inside the bold span.

Cleaned both into a single consistent **「ダッシュボードのインポート」** quoted term, matching the sibling **「ダッシュボード」** bold-quote convention already used earlier in the same step.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v8.5 (TiDB 8.5 versions)
  • v8.4 (TiDB 8.4 versions)
  • v8.3 (TiDB 8.3 versions)
  • v8.2 (TiDB 8.2 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)
  • v6.1 (TiDB 6.1 versions)
  • v5.4 (TiDB 5.4 versions)
  • v5.3 (TiDB 5.3 versions)

What is the related PR or file link(s)?

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

…shboard button

monitor-new-relic-integration.md had two pre-existing MT defects on
the 'Import dashboard' step (predating any of this session's UI-label
batches): one split the word ダッシュボード in half across a bold
marker boundary ('ダッシュ**ボードのインポート」**'), the other
trapped '[...] を' decoration inside the bold span
('**[ダッシュボードをインポート] を**'). Cleaned both into a single
consistent 「ダッシュボードのインポート」 quoted term, matching the
sibling 「ダッシュボード」 bold-quote convention already used earlier
in the same step.
@ti-chi-bot ti-chi-bot Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 17, 2026
@ti-chi-bot

ti-chi-bot Bot commented Aug 17, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@ti-chi-bot

ti-chi-bot Bot commented Aug 17, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign icemap for approval. For more information see the Code Review Process.
Please ensure that each of them provides their approval before proceeding.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 63b24ade-1e92-4e58-a888-efa9bdcbcf11

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ti-chi-bot ti-chi-bot Bot added missing-translation-status This PR does not have translation status info. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 17, 2026
@yahonda
yahonda marked this pull request as ready for review August 17, 2026 05:13
@ti-chi-bot ti-chi-bot Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 17, 2026
@qiancai qiancai added translation/no-need No need to translate this PR. and removed missing-translation-status This PR does not have translation status info. labels Aug 17, 2026
@ti-chi-bot ti-chi-bot Bot added the needs-1-more-lgtm Indicates a PR needs 1 more LGTM. label Aug 17, 2026
@ti-chi-bot

ti-chi-bot Bot commented Aug 17, 2026

Copy link
Copy Markdown

[LGTM Timeline notifier]

Timeline:

  • 2026-08-17 10:00:10.45045724 +0000 UTC m=+3645396.486552295: ☑️ agreed by qiancai.

@qiancai
qiancai merged commit 69b596a into pingcap:i18n-ja-release-8.5 Aug 17, 2026
5 of 9 checks passed
@yahonda
yahonda deleted the fix-new-relic-import-dashboard-bracket-2026-08-17 branch August 18, 2026 00:09
yahonda added a commit to yahonda/docs that referenced this pull request Aug 18, 2026
…onfirmed batch)

These 228 candidate labels were skipped by the mechanical 2-word batch
(pingcap#23535) because they couldn't be confirmed via a plain dbaas-ui grep
(mostly third-party console fields — AWS, Azure, Alibaba Cloud, GCP,
Kafka, Datadog, New Relic, n8n, Airbyte, PagerDuty, Slack — which
dbaas-ui doesn't cover, plus a handful of TiDB-native labels the
grep missed). Each was individually triaged against EN (release-8.5)
and, for TiDB-native items, dbaas-ui with a strict STRONG/WEAK/
COMMENT_ONLY/NO_HIT classifier, distinguishing genuine clickable UI
elements from prose/marketing/category-header bullets that merely
share the same words (the "No cost"/"High availability" false-positive
pattern from earlier batches).

265 occurrences fixed across 47 files. ~70 further candidates were
deliberately left in Japanese as category-header/concept-heading
prose, not clickable UI (same precedent as architecture-concepts.md).

Along the way this also fixed several pre-existing defects surfaced
by the same EN comparison: mismatched AWS IAM button-label pairs
(Attach Policies vs Attach Policy treated as the same word), particle-
and bracket-trapped-in-bold spans, a word-order/position-pairing
mis-mapping (Source/Target engine, Signing method/Account key,
Export Path), a mistranslation (一般公開→Public accessible), and an
orphaned bracket left over from an incomplete straddling-bracket fix.

"Import dashboard" (monitor-new-relic-integration.md) was explicitly
skipped — already handled by pingcap#23540.
yahonda added a commit to yahonda/docs that referenced this pull request Aug 19, 2026
…onfirmed batch)

These 228 candidate labels were skipped by the mechanical 2-word batch
(pingcap#23535) because they couldn't be confirmed via a plain dbaas-ui grep
(mostly third-party console fields — AWS, Azure, Alibaba Cloud, GCP,
Kafka, Datadog, New Relic, n8n, Airbyte, PagerDuty, Slack — which
dbaas-ui doesn't cover, plus a handful of TiDB-native labels the
grep missed). Each was individually triaged against EN (release-8.5)
and, for TiDB-native items, dbaas-ui with a strict STRONG/WEAK/
COMMENT_ONLY/NO_HIT classifier, distinguishing genuine clickable UI
elements from prose/marketing/category-header bullets that merely
share the same words (the "No cost"/"High availability" false-positive
pattern from earlier batches).

265 occurrences fixed across 47 files. ~70 further candidates were
deliberately left in Japanese as category-header/concept-heading
prose, not clickable UI (same precedent as architecture-concepts.md).

Along the way this also fixed several pre-existing defects surfaced
by the same EN comparison: mismatched AWS IAM button-label pairs
(Attach Policies vs Attach Policy treated as the same word), particle-
and bracket-trapped-in-bold spans, a word-order/position-pairing
mis-mapping (Source/Target engine, Signing method/Account key,
Export Path), a mistranslation (一般公開→Public accessible), and an
orphaned bracket left over from an incomplete straddling-bracket fix.

"Import dashboard" (monitor-new-relic-integration.md) was explicitly
skipped — already handled by pingcap#23540.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-1-more-lgtm Indicates a PR needs 1 more LGTM. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. translation/no-need No need to translate this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants