Skip to content

feat(i18n): add Slovenian (sl) to SUPPORTED_LOCALES (#29999) - #30006

Open
Rodrigoue9 wants to merge 1 commit into
calcom:mainfrom
Rodrigoue9:feat/bounty-29999
Open

feat(i18n): add Slovenian (sl) to SUPPORTED_LOCALES (#29999)#30006
Rodrigoue9 wants to merge 1 commit into
calcom:mainfrom
Rodrigoue9:feat/bounty-29999

Conversation

@Rodrigoue9

Copy link
Copy Markdown

Title

feat(i18n): add Slovenian (sl, sl-SI) to SUPPORTED_LOCALES (#29999)

Description

  • Adds sl and sl-SI to SUPPORTED_LOCALES in packages/platform/constants/api.ts.
  • Enables Slovenian language support and recognition across API endpoints and localization settings.

Closes #29999

@github-actions

Copy link
Copy Markdown
Contributor

Welcome to Cal.diy, @Rodrigoue9! Thanks for opening this pull request.

A few things to keep in mind:

  • This is Cal.diy, not Cal.com. Cal.diy is a community-driven, fully open-source fork of Cal.com licensed under MIT. Your changes here will be part of Cal.diy — they will not be deployed to the Cal.com production app.
  • Please review our Contributing Guidelines if you haven't already.
  • Make sure your PR title follows the Conventional Commits format.

A maintainer will review your PR soon. Thanks for contributing!

@github-actions github-actions Bot added ✨ feature New feature or request 🚨 needs approval This feature request has not been reviewed yet by the Product Team and needs approval beforehand labels Aug 19, 2026
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 2bc7a48a-57dc-4e6c-8237-f3254311160a

📥 Commits

Reviewing files that changed from the base of the PR and between 2b75ee0 and 385f3b6.

📒 Files selected for processing (4)
  • apps/api/v2/src/lib/enums/locales.ts
  • i18n.json
  • packages/dayjs/locales.ts
  • packages/i18n/locales/sl/common.json

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

Added Slovenian support by defining Locales.SL as "sl", adding sl to the locale targets, and importing dayjs/locale/sl in the Day.js locale aggregation module.

Merge Risk: ⚪ Minimal · up to 385f3

This change adds Slovenian localization support without any identified merge-blocking risk; it is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR adds locale registration only; it omits the required translation file, SUPPORTED_LOCALES updates, selection, and browser detection [#29999]. Add sl and sl-SI to SUPPORTED_LOCALES, add apps/web/public/static/locales/sl/common.json, and implement account selection and browser auto-detection.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: adding Slovenian support to the supported locale configuration.
Description check ✅ Passed The description directly concerns Slovenian locale support and matches the enum, i18n target, and Day.js localization changes.
Out of Scope Changes check ✅ Passed All reported changes support Slovenian locale registration and localization, so no unrelated code changes are evident.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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

Labels

✨ feature New feature or request 🚨 needs approval This feature request has not been reviewed yet by the Product Team and needs approval beforehand size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Slovenian language

1 participant