Skip to content

Translation: Add Korean (ko) language support and update translations#154

Open
d0ngjini wants to merge 2 commits intomattermost:mainfrom
d0ngjini:main
Open

Translation: Add Korean (ko) language support and update translations#154
d0ngjini wants to merge 2 commits intomattermost:mainfrom
d0ngjini:main

Conversation

@d0ngjini
Copy link

@d0ngjini d0ngjini commented Jan 6, 2026

Summary

This pull request updates the Korean translation files and enables Korean as a supported language in the application.

Changes:

  • Updated translation strings in webapp/i18n/ko.json to reflect the latest UI elements.
  • Registered ko (Korean) as a selectable language option in webapp/src/i18n.tsx and webapp/src/constants.ts.

Users can now select Korean from the language settings menu.

Ticket Link

N/A

image

Summary by CodeRabbit

  • New Features
    • Added full Korean language support to the application, including comprehensive translation updates across all UI strings, messages, and interface labels. Korean is now available as a selectable language option, providing users with a complete localized experience.

@marianunez
Copy link
Member

@coderabbitai review this PR

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

@marianunez I'll review this PR which adds Korean language support to the Mattermost Boards plugin. The changes appear to include:

  • Translation updates in webapp/i18n/ko.json
  • Korean language registration in webapp/src/i18n.tsx and webapp/src/constants.ts

I can see from the screenshot that Korean (한국어) is now available in the language selector and the UI is displaying Korean translations. Let me conduct a comprehensive review of the implementation.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

📝 Walkthrough

Walkthrough

Korean language support is added to the application through a new translation file, updated language constants configuration, and i18n module integration. Changes include translation data updates and configuration references to enable Korean as an available language option.

Changes

Cohort / File(s) Summary
Korean Localization
webapp/i18n/ko.json
New and updated Korean translations across UI strings, labels, messages, and action text. Includes new keys for admin badges and refinements to existing terminology for consistency.
Language Configuration
webapp/src/constants.ts, webapp/src/i18n.tsx
Adds Korean language entry to the supported languages list; imports Korean translation files and integrates language lookup in i18n message resolution.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (9 files):

⚔️ .github/workflows/ci.yml (content)
⚔️ .github/workflows/lint-server.yml (content)
⚔️ server/api/cards.go (content)
⚔️ webapp/i18n/ko.json (content)
⚔️ webapp/src/cardFilter.test.ts (content)
⚔️ webapp/src/cardFilter.ts (content)
⚔️ webapp/src/constants.ts (content)
⚔️ webapp/src/i18n.tsx (content)
⚔️ webapp/src/router.tsx (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main changes: adding Korean language support and updating translations, which is directly reflected in all three modified files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch main
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉


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

Copy link
Member

@marianunez marianunez left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @d0ngjini!

@marianunez
Copy link
Member

@d0ngjini can you check the snapshots that need to be udpated as the webapp tests are failing?

@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants