Skip to content

feat(localizations): Update is-IS, nb-NO, fi-FI, hr-HR, hu-HU with missing translations#8224

Open
bjaben wants to merge 4 commits intoclerk:mainfrom
bjaben:feat/update-nordic-and-european-localizations
Open

feat(localizations): Update is-IS, nb-NO, fi-FI, hr-HR, hu-HU with missing translations#8224
bjaben wants to merge 4 commits intoclerk:mainfrom
bjaben:feat/update-nordic-and-european-localizations

Conversation

@bjaben
Copy link
Copy Markdown
Contributor

@bjaben bjaben commented Apr 1, 2026

Description

Update five community-maintained localizations with complete translations for all previously missing sections. This brings these languages up to parity with en-US.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • Localization
    • Expanded and improved translations for Icelandic, Norwegian (Bokmål), Finnish, Croatian, and Hungarian.
    • Filled many previously missing strings across billing, subscriptions, pricing/seats, API key management, authentication flows, MFA/passkey setup, reverification, reset-password tasks, and error messages.
    • Enhanced UI text for account/organization screens, checkout/plan pages, and membership/seat alerts for clearer user experience.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026

@bjaben is attempting to deploy a commit to the Clerk Production Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 1, 2026

🦋 Changeset detected

Latest commit: 945c91f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@clerk/localizations Patch
@clerk/ui Patch
@clerk/chrome-extension Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: db51ba47-f026-4558-87ae-4c6cc7595b82

📥 Commits

Reviewing files that changed from the base of the PR and between 241ee6c and 2a053ff.

📒 Files selected for processing (5)
  • packages/localizations/src/fi-FI.ts
  • packages/localizations/src/hr-HR.ts
  • packages/localizations/src/hu-HU.ts
  • packages/localizations/src/is-IS.ts
  • packages/localizations/src/nb-NO.ts

📝 Walkthrough

Walkthrough

A changeset file .changeset/update-icelandic-localization.md is added to mark a patch release for the @clerk/localizations package. Five localization resource files (is-IS, nb-NO, fi-FI, hr-HR, hu-HU) are updated: many previously undefined entries were replaced with concrete translated strings across billing, API key management, authentication and re-verification flows, MFA/setup, organization and profile UIs, error messages, and templated translations with interpolation and locale-specific formatting.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating five localization files (is-IS, nb-NO, fi-FI, hr-HR, hu-HU) with missing translations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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 and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
packages/localizations/src/hu-HU.ts (1)

221-221: ⚠️ Potential issue | 🟠 Major

Use the numeric formatter for dates.numeric.

Line 221 uses timeString('hu-HU') for dates.numeric, so this entry will render only the time instead of a numeric date. That breaks the Hungarian dates.numeric variant anywhere it is used.

Suggested fix
-    numeric: "{{ date | timeString('hu-HU') }}",
+    numeric: "{{ date | numeric('hu-HU') }}",

As per coding guidelines, "Check for formatting issues, such as missing placeholders, incorrect variable usage, or syntax errors."

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@packages/localizations/src/hu-HU.ts` at line 221, The dates.numeric entry
currently uses timeString('hu-HU') which outputs only the time; update the
dates.numeric formatter to use the numeric date formatter instead of timeString
(replace the timeString('hu-HU') usage in the dates.numeric entry with the
appropriate numeric formatter for Hungarian, e.g., the project’s
numeric/dateNumeric formatter for 'hu-HU') so that dates.numeric renders a full
numeric date rather than only the time.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@packages/localizations/src/hu-HU.ts`:
- Line 221: The dates.numeric entry currently uses timeString('hu-HU') which
outputs only the time; update the dates.numeric formatter to use the numeric
date formatter instead of timeString (replace the timeString('hu-HU') usage in
the dates.numeric entry with the appropriate numeric formatter for Hungarian,
e.g., the project’s numeric/dateNumeric formatter for 'hu-HU') so that
dates.numeric renders a full numeric date rather than only the time.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: 8c700ab2-bf6a-4872-9b40-fde18dca2e45

📥 Commits

Reviewing files that changed from the base of the PR and between 82b02eb and 241ee6c.

📒 Files selected for processing (6)
  • .changeset/update-icelandic-localization.md
  • packages/localizations/src/fi-FI.ts
  • packages/localizations/src/hr-HR.ts
  • packages/localizations/src/hu-HU.ts
  • packages/localizations/src/is-IS.ts
  • packages/localizations/src/nb-NO.ts

Copy link
Copy Markdown
Member

@Ephem Ephem left a comment

Choose a reason for hiding this comment

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

Thanks a lot for this contribution, this is super valuable! 🙏

Based on a few agent reviews and my own reading of the Norwegian translations (I don't master the language at all, but as a Swedish speaker I can read it), this is looking great.

Quick background question: How many of these languages do you master or have familiarity with? I get if this is mostly machine translated, which is more than fine, but how many of these languages would you say you can meaningfully judge the quality of yourself?

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 2, 2026

Open in StackBlitz

@clerk/agent-toolkit

npm i https://pkg.pr.new/@clerk/agent-toolkit@8224

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@8224

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@8224

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@8224

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@8224

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@8224

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@8224

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@8224

@clerk/express

npm i https://pkg.pr.new/@clerk/express@8224

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@8224

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@8224

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@8224

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@8224

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@8224

@clerk/react

npm i https://pkg.pr.new/@clerk/react@8224

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@8224

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@8224

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@8224

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@8224

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@8224

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@8224

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@8224

commit: 2a053ff

@bjaben
Copy link
Copy Markdown
Contributor Author

bjaben commented Apr 2, 2026

Thanks a lot for this contribution, this is super valuable! 🙏

Based on a few agent reviews and my own reading of the Norwegian translations (I don't master the language at all, but as a Swedish speaker I can read it), this is looking great.

Quick background question: How many of these languages do you master or have familiarity with? I get if this is mostly machine translated, which is more than fine, but how many of these languages would you say you can meaningfully judge the quality of yourself?

Our team covers IS, NO and FI pretty well but less so HR and HU.

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