-
Notifications
You must be signed in to change notification settings - Fork 437
chore(clerk-js,localizations): Update TaskChooseOrganization copies for create organization screen
#6584
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(clerk-js,localizations): Update TaskChooseOrganization copies for create organization screen
#6584
Changes from all commits
c4ac648
cbec725
f85a558
3623bbc
db9f39d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| --- | ||
| '@clerk/localizations': patch | ||
| '@clerk/clerk-js': patch | ||
| '@clerk/testing': patch | ||
| '@clerk/types': patch | ||
| --- | ||
|
|
||
| Update copies for create organization screen on session tasks |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -813,6 +813,10 @@ export const daDK: LocalizationResource = { | |
| createOrganization: { | ||
| formButtonReset: undefined, | ||
| formButtonSubmit: undefined, | ||
| formFieldInputPlaceholder__name: undefined, | ||
| formFieldInputPlaceholder__slug: undefined, | ||
| formFieldLabel__name: undefined, | ||
| formFieldLabel__slug: undefined, | ||
|
Comment on lines
+816
to
+819
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainProvide Danish labels/placeholders for create-organization fields in the task flow. These are the keys directly tied to the PR objective. Recommend aligning with existing translations used elsewhere (e.g., organizationName/organizationSlug) for consistency. Apply this diff: - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,
+ formFieldInputPlaceholder__name: 'Indtast organisationens navn',
+ formFieldInputPlaceholder__slug: 'Indtast slug URL',
+ formFieldLabel__name: 'Navn',
+ formFieldLabel__slug: 'Slug URL',To verify the new keys exist in the base locale and are consumed consistently, run: 🏁 Script executed: #!/bin/bash
set -euo pipefail
# Locate files
EN_FILE="$(fd -t f 'en-US.ts' | head -n1)"
DK_FILE="$(fd -t f 'da-DK.ts' | head -n1)"
if [[ -z "${EN_FILE:-}" || -z "${DK_FILE:-}" ]]; then
echo "Could not locate en-US.ts or da-DK.ts" >&2
exit 1
fi
echo "en-US: $EN_FILE"
echo "da-DK: $DK_FILE"
echo
# Keys to verify across locales
keys=(
'cancelFreeTrial:'
'cancelFreeTrialDescription:'
'cancelFreeTrialTitle:'
'keepFreeTrial:'
'firstPaymentAmount:'
'firstPaymentOn:'
'trialEndsOn:'
'trialStartedOn:'
'formFieldInputPlaceholder__name:'
'formFieldInputPlaceholder__slug:'
'formFieldLabel__name:'
'formFieldLabel__slug:'
)
missing=false
for k in "${keys[@]}"; do
echo "=== Key: $k"
rg -nP "$k" "$EN_FILE" -C1 || { echo "Missing in en-US: $k"; missing=true; }
rg -nP "$k" "$DK_FILE" -n || { echo "Missing in da-DK: $k"; missing=true; }
echo
done
$missing && { echo "One or more keys missing. Please reconcile."; exit 1; } || echo "All keys present in en-US and da-DK."Length of output: 6208 |
||
| subtitle: undefined, | ||
| title: undefined, | ||
| }, | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -826,6 +826,10 @@ export const deDE: LocalizationResource = { | |||||||||
| createOrganization: { | ||||||||||
| formButtonReset: undefined, | ||||||||||
| formButtonSubmit: undefined, | ||||||||||
| formFieldInputPlaceholder__name: undefined, | ||||||||||
| formFieldInputPlaceholder__slug: undefined, | ||||||||||
| formFieldLabel__name: undefined, | ||||||||||
| formFieldLabel__slug: undefined, | ||||||||||
|
Comment on lines
+829
to
+832
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainDo not set new create-organization field keys to Explicit Preferred quick fix: remove to allow fallback - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,Alternative: add translations now - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,
+ formFieldInputPlaceholder__name: 'Name der Organisation eingeben',
+ formFieldInputPlaceholder__slug: 'my-org',
+ formFieldLabel__name: 'Name',
+ formFieldLabel__slug: 'Slug',🏁 Script executed: #!/bin/bash
set -euo pipefail
# 1) Confirm en-US has the new keys for fallback
fd -a en-US.ts | while read -r f; do
echo "Checking keys in $f"
rg -n -C2 "taskChooseOrganization|formFieldLabel__name|formFieldInputPlaceholder__name|formFieldLabel__slug|formFieldInputPlaceholder__slug" "$f" || true
done
# 2) Inspect localization merge to verify that explicit `undefined` overrides base strings
rg -n -C3 "parseLocalizationResource|fastDeepMerge" packages -g '!**/dist/**' -g '!**/build/**' || trueLength of output: 16516 Remove undefined create-organization keys in de-DE localization Explicit • File: packages/localizations/src/de-DE.ts Preferred quick fix – allow fallback: - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,Alternative – add German translations: - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,
+ formFieldInputPlaceholder__name: 'Name der Organisation eingeben',
+ formFieldInputPlaceholder__slug: 'my-org',
+ formFieldLabel__name: 'Name',
+ formFieldLabel__slug: 'Slug',📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||
| subtitle: undefined, | ||||||||||
| title: undefined, | ||||||||||
| }, | ||||||||||
|
|
||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -822,6 +822,10 @@ export const esCR: LocalizationResource = { | |||||||||||||||||
| createOrganization: { | ||||||||||||||||||
| formButtonReset: undefined, | ||||||||||||||||||
| formButtonSubmit: undefined, | ||||||||||||||||||
| formFieldInputPlaceholder__name: undefined, | ||||||||||||||||||
| formFieldInputPlaceholder__slug: undefined, | ||||||||||||||||||
| formFieldLabel__name: undefined, | ||||||||||||||||||
| formFieldLabel__slug: undefined, | ||||||||||||||||||
|
Comment on lines
+825
to
+828
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainPopulate create-organization labels/placeholders for the session task (suggested es-CR) Setting these avoids en-US fallback in the new Create Organization screen. Apply: - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,
+ formFieldInputPlaceholder__name: 'Mi organización',
+ formFieldInputPlaceholder__slug: 'mi-organizacion',
+ formFieldLabel__name: 'Nombre',
+ formFieldLabel__slug: 'Apodo',To verify usage aligns with the new path: 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "Check UI references the new taskChooseOrganization.createOrganization keys"
rg -nP "taskChooseOrganization\.createOrganization\.(formField(Label|InputPlaceholder)__(name|slug))" packages/clerk-js -C2Length of output: 2075 Populate es-CR translations for Create Organization form fields To prevent falling back to en-US in the new Create Organization screen, add the Spanish (Costa Rica) labels and placeholders for the File: packages/localizations/src/es-CR.ts (around lines 825–828) - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,
+ formFieldInputPlaceholder__name: 'Mi organización',
+ formFieldInputPlaceholder__slug: 'mi-organizacion',
+ formFieldLabel__name: 'Nombre',
+ formFieldLabel__slug: 'Apodo',📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||
| subtitle: undefined, | ||||||||||||||||||
| title: undefined, | ||||||||||||||||||
| }, | ||||||||||||||||||
|
|
||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -816,6 +816,10 @@ export const esES: LocalizationResource = { | |
| createOrganization: { | ||
| formButtonReset: undefined, | ||
| formButtonSubmit: undefined, | ||
| formFieldInputPlaceholder__name: undefined, | ||
| formFieldInputPlaceholder__slug: undefined, | ||
| formFieldLabel__name: undefined, | ||
| formFieldLabel__slug: undefined, | ||
|
Comment on lines
+819
to
+822
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainLGTM on key naming/location; values need completion The new keys are correctly scoped under Run the following to verify type coverage and en-US baseline values for these keys: 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "Check that types include the new keys:"
rg -n "formField(InputPlaceholder|Label)__name|formField(InputPlaceholder|Label)__slug" packages/types/src/localization.ts -C3 --type=ts
echo
echo "Show en-US values for the same keys (should be defined):"
rg -n "formField(InputPlaceholder|Label)__name|formField(InputPlaceholder|Label)__slug" packages/localizations/src/en-US.ts -C3 --type=ts
echo
echo "Scan all locales for missing values on the new keys (undefined):"
rg -nP --type=ts "taskChooseOrganization[\s\S]*createOrganization[\s\S]*formField(InputPlaceholder|Label)__(name|slug)\s*:\s*undefined" packages/localizations/src | sed 's/^/undefined: /'
echo
echo "Optional: list locales that already translated them (non-undefined):"
rg -nP --type=ts "taskChooseOrganization[\s\S]*createOrganization[\s\S]*formField(InputPlaceholder|Label)__(name|slug)\s*:\s*'[^']+'" packages/localizations/src | sed 's/^/translated: /'Length of output: 1731 Complete missing translations for new localization keys The new keys are correctly defined in the types and the en-US locale provides baseline values. Please supply translations for these keys in all non-en locale files—currently they’re left as • packages/localizations/src/es-ES.ts (lines 819–822):
Ensure no locale leaves these values undefined. 🤖 Prompt for AI Agents |
||
| subtitle: undefined, | ||
| title: undefined, | ||
| }, | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -807,6 +807,10 @@ export const kkKZ: LocalizationResource = { | |||||||||
| createOrganization: { | ||||||||||
| formButtonReset: undefined, | ||||||||||
| formButtonSubmit: undefined, | ||||||||||
| formFieldInputPlaceholder__name: undefined, | ||||||||||
| formFieldInputPlaceholder__slug: undefined, | ||||||||||
| formFieldLabel__name: undefined, | ||||||||||
| formFieldLabel__slug: undefined, | ||||||||||
|
Comment on lines
+810
to
+813
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainAvoid Provide Kazakh strings or remove these keys to inherit en-US. Preferred quick fix: remove to allow fallback - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,Alternative: add Kazakh translations - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,
+ formFieldInputPlaceholder__name: 'Ұйым атауын енгізіңіз',
+ formFieldInputPlaceholder__slug: 'my-org',
+ formFieldLabel__name: 'Атауы',
+ formFieldLabel__slug: 'Slug',🏁 Script executed: #!/bin/bash
set -euo pipefail
# 1) Confirm en-US has the keys present for fallback
fd -a en-US.ts | while read -r f; do
rg -n -C2 "taskChooseOrganization|formFieldLabel__name|formFieldInputPlaceholder__name|formFieldLabel__slug|formFieldInputPlaceholder__slug" "$f" || true
done
# 2) Inspect merge behavior to ensure undefined overrides base (helps decide remove vs translate)
rg -n -C3 "parseLocalizationResource|fastDeepMerge" packages -g '!**/dist/**' -g '!**/build/**' || trueLength of output: 16363 Remove undefined overrides in kk-KZ localization The four • File: packages/localizations/src/kk-KZ.ts (lines 810–813) - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,Alternative – add Kazakh translations: - formFieldInputPlaceholder__name: undefined,
- formFieldInputPlaceholder__slug: undefined,
- formFieldLabel__name: undefined,
- formFieldLabel__slug: undefined,
+ formFieldInputPlaceholder__name: 'Ұйым атауын енгізіңіз',
+ formFieldInputPlaceholder__slug: 'my-org',
+ formFieldLabel__name: 'Атауы',
+ formFieldLabel__slug: 'Slug',📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||
| subtitle: undefined, | ||||||||||
| title: undefined, | ||||||||||
| }, | ||||||||||
|
|
||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
LGTM: localization keys correctly migrated to the new taskChooseOrganization path.
Run this quick check to ensure:
If you want, I can add a lightweight test that renders this screen with a mock localization to assert label/placeholder usage.
🏁 Script executed:
Length of output: 765
Missing localization key definitions in
en-US.tsand typesThe new keys referenced in
CreateOrganizationScreen.tsxaren’t defined in the localization files or the type surface, so they’ll break at runtime and in typing:• packages/localizations/src/en-US.ts is missing:
taskChooseOrganization.createOrganization.formFieldLabel__nametaskChooseOrganization.createOrganization.formFieldInputPlaceholder__nametaskChooseOrganization.createOrganization.formFieldLabel__slugtaskChooseOrganization.createOrganization.formFieldInputPlaceholder__slug• packages/types/src (your localization key union) needs to include those same four keys.
Suggested additions:
— packages/localizations/src/en-US.ts
— packages/types/src/index.ts (or wherever LocalizationKeys is defined)
Once those are in place, the labels and placeholders will resolve correctly.
🤖 Prompt for AI Agents