Skip to content

Conversation

@ArmandPhilippot
Copy link
Member

@ArmandPhilippot ArmandPhilippot commented Jan 20, 2026

Description

Adds changes from #194 to the French translations in start-here and to package-catalog/index.mdx.

Summary by CodeRabbit

  • Documentation
    • Removed update badges from French documentation pages.
    • Updated libSQL client package recommendations.
    • Enhanced environment variables documentation with new optional configuration parameters for libSQL, MySQL, and PostgreSQL.
    • Added comprehensive storage API configuration guide with S3 example and setup instructions.
    • Updated version references in getting started guide.

✏️ Tip: You can customize this high-level summary in your review settings.

@ArmandPhilippot ArmandPhilippot requested a review from a team as a code owner January 20, 2026 11:24
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 20, 2026

Warning

Rate limit exceeded

@ArmandPhilippot has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 5 minutes and 0 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 010e73a and b40ebfc.

📒 Files selected for processing (2)
  • src/content/docs/fr/package-catalog/index.mdx
  • src/content/docs/fr/start-here/getting-started.mdx
📝 Walkthrough

Walkthrough

French start-here documentation files updated: badge metadata removed from configuration and why-studioCMS frontmatter, environment variables reorganized with new libSQL options documented, version references updated in getting-started, and storage API configuration guidance added.

Changes

Cohort / File(s) Summary
Badge removal from frontmatter
src/content/docs/fr/start-here/configuration.mdx, src/content/docs/fr/start-here/why-studioCMS.mdx
Removed sidebar badge blocks ("Mis à jour" with success variant) from MDX front matter metadata.
Environment variables documentation
src/content/docs/fr/start-here/environment-variables.mdx
Reorganized libSQL section: commented out sync options, added commented optional variables (encryption key, read-your-writes, offline mode, TLS, concurrency). Added connection limit variables for MySQL and PostgreSQL sections.
Version and configuration updates
src/content/docs/fr/start-here/getting-started.mdx
Updated version reference (0.1.0-beta.31 → v0.1.0), updated libSQL package name (@libsql/kysely-libsql → kysely-turso), and added comprehensive storage API configuration section with examples.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • withstudiocms/docs#186: Modifies same French start-here files with overlapping badge removal and environment variable content updates
  • withstudiocms/docs#153: Removes identical "badge" metadata from French MDX frontmatter across multiple files
  • withstudiocms/docs#154: Removes the same badge block from src/content/docs/fr/start-here/configuration.mdx

Suggested labels

i18n

Suggested reviewers

  • dreyfus92
  • Adammatthiesen

Poem

🐰 Badges fade like morning dew,
French docs dance now fresh and new,
Storage configs, libSQL's tune,
Version bumps beneath the moon! 🌙

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: updating French translation files in the start-here documentation section, which is reflected in all four modified French MDX files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@studiocms-no-reply
Copy link
Member

studiocms-no-reply commented Jan 20, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
fr/package-catalog/index.mdx Localization changed, will be marked as complete.
fr/start-here/configuration.mdx Localization changed, will be marked as complete.
fr/start-here/environment-variables.mdx Localization changed, will be marked as complete.
fr/start-here/getting-started.mdx Localization changed, will be marked as complete.
fr/start-here/why-studioCMS.mdx Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@src/content/docs/fr/start-here/getting-started.mdx`:
- Around line 377-424: The French page contains broken links to missing French
docs; update the three link targets in
src/content/docs/fr/start-here/getting-started.mdx so they point to existing
content (either add French translations or change the hrefs to the English
counterparts). Specifically, fix the ReadMore link currently pointing to
"/fr/storage-api/" to "/en/storage-api/", change the ReadMore link to
"/fr/package-catalog#storage-managers" to "/en/package-catalog#storage-managers"
(or to the correct catalog anchor), and update the ReadMore link to
"/fr/package-catalog/storage-managers/s3-storage/#environment-variable-examples"
to
"/en/package-catalog/storage-managers/s3-storage/#environment-variable-examples";
alternatively, if you add translations, create the missing French files
(storage-api and package-catalog/storage-managers/s3-storage) and keep the
original /fr/* paths.
🧹 Nitpick comments (1)
src/content/docs/fr/start-here/environment-variables.mdx (1)

38-44: Add placeholders for sync interval/URL to avoid ambiguity.
The two new commented vars omit any hint of expected format. Adding lightweight placeholders would align with the rest of the block and reduce guesswork.

💡 Suggested tweak
-# CMS_LIBSQL_SYNC_INTERVAL= (facultatif)
-# CMS_LIBSQL_SYNC_URL= (facultatif)
+# CMS_LIBSQL_SYNC_INTERVAL=<number> (facultatif)
+# CMS_LIBSQL_SYNC_URL=<url> (facultatif)

@ArmandPhilippot ArmandPhilippot marked this pull request as draft January 20, 2026 11:29
@ArmandPhilippot ArmandPhilippot marked this pull request as ready for review January 20, 2026 11:32
Copy link
Member

@dreyfus92 dreyfus92 left a comment

Choose a reason for hiding this comment

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

Thank you @ArmandPhilippot 👀

@dreyfus92 dreyfus92 merged commit 78e7e48 into withstudiocms:main Jan 20, 2026
9 checks passed
@ArmandPhilippot ArmandPhilippot deleted the i18n/fr-update-start-here branch January 20, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants