Skip to content

Enable Indonesian (id) as an in-app locale - #715

Merged
krokosik merged 3 commits into
mainfrom
copilot/enable-indonesian-locale
Aug 1, 2026
Merged

Enable Indonesian (id) as an in-app locale#715
krokosik merged 3 commits into
mainfrom
copilot/enable-indonesian-locale

Conversation

Copilot AI commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Description

Indonesian translations already existed from Weblate, but id was not enabled in runtime locale/language config. This PR enables id end-to-end so it is routable, selectable in language pickers, and accepted by locale-switching validation.

  • Locale registration

    • Added id to i18n.locales in next-i18next.config.js.
  • Language metadata + picker source

    • Added Indonesian to getSupportedLanguages() in src/utils/i18n/client.ts:
      { code: 'id', name: 'Bahasa Indonesia' }
    • This feeds both language picker UIs and /api/locale allowed locale validation.
  • Regression coverage

    • Added src/tests/i18nClient.test.ts to assert Indonesian is present in supported language metadata.
  • Testing notes

    • Included focused unit coverage for locale-list enablement behavior.

Demo

N/A (configuration/runtime locale enablement change)

Checklist

  • I have read CONTRIBUTING.md in its entirety
  • I have performed a self-review of my own code
  • I have added unit tests to cover my changes
  • The last commit successfully passed pre-commit checks
  • Any AI code was thoroughly reviewed by me

Summary by CodeRabbit

  • New Features
    • Added Indonesian as a supported language.
    • Indonesian is now available for localized application content.

Copilot AI changed the title [WIP] Add Indonesian as an app locale Enable Indonesian (id) as an in-app locale Aug 1, 2026
Copilot AI requested a review from krokosik August 1, 2026 12:41
Comment thread src/tests/i18nClient.test.ts Outdated
@@ -0,0 +1,10 @@
import { getSupportedLanguages } from '~/utils/i18n/client';

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@copilot remove this test

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done in b599f61.

@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d99c0d06-a9bb-4846-8457-98d3fd8a3106

📥 Commits

Reviewing files that changed from the base of the PR and between a3b4dc8 and b599f61.

📒 Files selected for processing (2)
  • next-i18next.config.js
  • src/utils/i18n/client.ts

📝 Walkthrough

Walkthrough

The change adds Indonesian (id) to the configured i18n locales and to the supported language list. The display name is Bahasa Indonesia.

Changes

Indonesian locale support

Layer / File(s) Summary
Register Indonesian locale
next-i18next.config.js, src/utils/i18n/client.ts
The i18n configuration includes id. The supported language list includes id with the display name Bahasa Indonesia.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: krokosik

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch copilot/enable-indonesian-locale

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

Copilot AI requested a review from krokosik August 1, 2026 12:48
@krokosik
krokosik marked this pull request as ready for review August 1, 2026 13:26
@krokosik
krokosik merged commit 7e6a401 into main Aug 1, 2026
6 checks passed
@krokosik
krokosik deleted the copilot/enable-indonesian-locale branch August 1, 2026 13:26
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