Uh oh!
There was an error while loading. Please reload this page.
fix(localizations): Update tr-TR waitlist translations - #9410
fix(localizations): Update tr-TR waitlist translations#9410emirmuminoglu wants to merge 2 commits into
Conversation
Waitlist is linking to sign in page but the action text is "Still don't have an account?", and action link is "Join Waitlist"
|
@emirmuminoglu is attempting to deploy a commit to the Clerk Production Team on Vercel. A member of the Team first needs to authorize it. |
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (1)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughUpdated two Turkish localization strings. The email placeholder now uses Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk:🔵 Low · up to The Turkish waitlist prompt links to sign-in but still asks whether the user has an account, which may remain misleading. This is a bounded, non-blocking wording issue that should be corrected or explicitly accepted before merge. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
Warning Linked repositories: Your configuration references 7 linked repositories, but your current plan allows 5. Analyzed Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/localizations/src/tr-TR.ts`:
- Line 2222: Update the waitlist prompt’s actionText in the Turkish localization
to match the base resource’s “Already have access?” intent by using “Zaten
erişiminiz var mı?”.
- Line 2221: Update the actionLink value in the Turkish localization entry to
use sentence case, changing “Giriş Yap” to “Giriş yap” while leaving the
surrounding localization unchanged.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 6e903be4-b5f9-414e-b471-186a4d8a1612
📒 Files selected for processing (1)
packages/localizations/src/tr-TR.ts
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
clerk/clerk_go(manual)clerk/dashboard(manual)clerk/accounts(manual)clerk/backoffice(manual)clerk/clerk(manual)clerk/clerk-docs(manual) → reviewed against branchpatch-1instead of the default branchclerk/cloudflare-workers(manual)
Uh oh!
There was an error while loading. Please reload this page.
| actionLink: 'Bekleme listesine katıl', | ||
| actionText: 'Hala bir hesabınız yok mu?', | ||
| actionLink: 'Giriş Yap', | ||
| actionText: 'Hesabınız var mı?', |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Match the waitlist prompt to the base localization intent.
The base resource uses Already have access?, but Hesabınız var mı? asks whether the user has an account. These are different conditions. Use Zaten erişiminiz var mı? unless this locale-specific wording is intentional. (github.com)
As per path instructions: “Use the most up-to-date base localization file to validate changes, ensuring consistency and completeness.”
Proposed fix
- actionText: 'Hesabınız var mı?',+ actionText: 'Zaten erişiminiz var mı?',📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| actionText: 'Hesabınız var mı?', | |
| actionText: 'Zaten erişiminiz var mı?', |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@packages/localizations/src/tr-TR.ts` at line 2222, Update the waitlist
prompt’s actionText in the Turkish localization to match the base resource’s
“Already have access?” intent by using “Zaten erişiminiz var mı?”.
Source: Path instructions
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Description
Waitlist actions were misleading. Action links to sign in page but the action text was "Still don't have an account?", and action link was "Join Waitlist". I changed them to match the behavior.
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change