Skip to content

Dynamically list languages#192

Merged
bl-ue merged 1 commit intomainfrom
dynamically-list-langs
Mar 8, 2026
Merged

Dynamically list languages#192
bl-ue merged 1 commit intomainfrom
dynamically-list-langs

Conversation

@bl-ue
Copy link
Member

@bl-ue bl-ue commented Mar 8, 2026

Summary by CodeRabbit

  • Refactor
    • Updated the language selector in Settings to dynamically load available languages from configuration instead of using a fixed list. This enables more flexible language option management without requiring code changes.

@bl-ue bl-ue enabled auto-merge (squash) March 8, 2026 00:12
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8495964e-3538-460a-b020-842979c47641

📥 Commits

Reviewing files that changed from the base of the PR and between c2fef8b and 1f1ee09.

📒 Files selected for processing (1)
  • frontend/src/components/common/SettingsDialog.tsx

📝 Walkthrough

Walkthrough

The language selector in the settings dialog now dynamically renders language options from a centralized i18n configuration instead of using hardcoded values, while maintaining the same user-facing functionality and control flow.

Changes

Cohort / File(s) Summary
Dynamic Language Configuration
frontend/src/components/common/SettingsDialog.tsx
Replaced hardcoded language options with dynamic rendering from supportedLanguages array, using languageNames for display labels. Maps over configuration to generate SelectItem components instead of maintaining a static list.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hardcoded lists hop away,
Dynamic config saves the day,
Language options flow so free,
From configuration, elegantly! ✨

🚥 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 'Dynamically list languages' accurately and specifically describes the main change: replacing hardcoded language options with dynamic rendering from supportedLanguages.
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.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dynamically-list-langs

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@bl-ue bl-ue merged commit 8dd3c30 into main Mar 8, 2026
2 checks passed
@bl-ue bl-ue deleted the dynamically-list-langs branch March 8, 2026 00:16
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.

1 participant