Skip to content

fix(FR-2329): improve bulk user creation max count message wording#6224

Open
yomybaby wants to merge 1 commit intomainfrom
03-27-fix_fr-2329_improve_bulk_user_creation_max_count_message_wording
Open

fix(FR-2329): improve bulk user creation max count message wording#6224
yomybaby wants to merge 1 commit intomainfrom
03-27-fix_fr-2329_improve_bulk_user_creation_max_count_message_wording

Conversation

@yomybaby
Copy link
Copy Markdown
Member

@yomybaby yomybaby commented Mar 27, 2026

Resolves #6050(FR-2329)

Summary

  • Updated credential.validation.MaxUserCount i18n key in en.json and ko.json to clarify that the limit applies to users created "at once"
  • Added the missing MaxUserCount key to all 19 other language files with appropriate translations

Test plan

  • Verify the bulk user creation modal shows the improved wording in English
  • Verify Korean translation displays correctly
  • Spot-check a few other languages to confirm the key is present and renders properly

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings March 27, 2026 04:21
@github-actions github-actions bot added area:ux UI / UX issue. area:i18n Localization labels Mar 27, 2026
@github-actions github-actions bot added the size:S 10~30 LoC label Mar 27, 2026
Copy link
Copy Markdown
Member Author


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the bulk user creation “max count” validation message to be clearer, and adds/aligns the corresponding translation key across all supported locales in resources/i18n/.

Changes:

  • Reworded the English and Korean credential.validation.MaxUserCount message to clarify it applies “at once”.
  • Added credential.validation.MaxUserCount translations for all other locale JSON files.

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated no comments.

Show a summary per file
File Description
resources/i18n/en.json Reworded credential.validation.MaxUserCount for clarity.
resources/i18n/ko.json Reworded credential.validation.MaxUserCount for clarity.
resources/i18n/de.json Added translation for credential.validation.MaxUserCount.
resources/i18n/el.json Added translation for credential.validation.MaxUserCount.
resources/i18n/es.json Added translation for credential.validation.MaxUserCount.
resources/i18n/fi.json Added translation for credential.validation.MaxUserCount.
resources/i18n/fr.json Added translation for credential.validation.MaxUserCount.
resources/i18n/id.json Added translation for credential.validation.MaxUserCount.
resources/i18n/it.json Added translation for credential.validation.MaxUserCount.
resources/i18n/ja.json Added translation for credential.validation.MaxUserCount.
resources/i18n/mn.json Added translation for credential.validation.MaxUserCount.
resources/i18n/ms.json Added translation for credential.validation.MaxUserCount.
resources/i18n/pl.json Added translation for credential.validation.MaxUserCount.
resources/i18n/pt.json Added translation for credential.validation.MaxUserCount.
resources/i18n/pt-BR.json Added translation for credential.validation.MaxUserCount.
resources/i18n/ru.json Added translation for credential.validation.MaxUserCount.
resources/i18n/th.json Added translation for credential.validation.MaxUserCount.
resources/i18n/tr.json Added translation for credential.validation.MaxUserCount.
resources/i18n/vi.json Added translation for credential.validation.MaxUserCount.
resources/i18n/zh-CN.json Added translation for credential.validation.MaxUserCount.
resources/i18n/zh-TW.json Added translation for credential.validation.MaxUserCount.

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

Labels

area:i18n Localization area:ux UI / UX issue. size:S 10~30 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve bulk user creation max count message wording

2 participants