Skip to content

i18n(fr): move utils/kysely-and-sdk.mdx to ecosystem/packages#205

Merged
dreyfus92 merged 3 commits into
withstudiocms:mainfrom
ArmandPhilippot:i18n/fr-move-kysely-sdk
Jan 24, 2026
Merged

i18n(fr): move utils/kysely-and-sdk.mdx to ecosystem/packages#205
dreyfus92 merged 3 commits into
withstudiocms:mainfrom
ArmandPhilippot:i18n/fr-move-kysely-sdk

Conversation

@ArmandPhilippot
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot commented Jan 24, 2026

Description

Updates the French translations with changes from #194 to

  • Remove utils/kysely-and-sdk.mdx
  • Add ecosystem/packages/kysely.mdx and ecosystem/packages/sdk.mdx

Summary by CodeRabbit

Release Notes

  • Documentation
    • Restructured French docs to present a Kysely-based database client with updated metadata, focused narrative, and revised code examples and guidance.
    • Added a new French SDK documentation page outlining core SDK features (auth, secure config, CRUD patterns, caching, middleware, API helpers) and links to further resources.

✏️ 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 24, 2026 12:51
@studiocms-no-reply
Copy link
Copy Markdown
Member

studiocms-no-reply commented Jan 24, 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/ecosystem/packages/kysely.mdx Localization added, will be marked as complete.
fr/ecosystem/packages/sdk.mdx Localization added, 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.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 24, 2026

Warning

Rate limit exceeded

@ArmandPhilippot has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 12 minutes and 28 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.

📝 Walkthrough

Walkthrough

Restructures the French Kysely package doc to focus on a Kysely-based StudioCMS DB client with updated imports and examples; adds a new French SDK documentation page describing SDK features and navigation links. All changes are documentation-only.

Changes

Cohort / File(s) Summary
Kysely doc updates
src/content/docs/fr/ecosystem/packages/kysely.mdx
Frontmatter updated (title, type, catalogEntry, description, sidebar badge); removed internal usage and SDK overview; refactored code samples to import tables from @withstudiocms/sdk/tables; adjusted headings, added caution Aside, fixed ReadMore links; one debug annotation inserted.
New SDK documentation
src/content/docs/fr/ecosystem/packages/sdk.mdx
Added new French SDK MDX page with frontmatter, overview and detailed feature list (auth, config, CRUD/Error handling, modules, cache, middlewares, diff tracking, REST helpers, notifications, token bucket) and navigation/ReadMore sections.

Sequence Diagram(s)

(omitted — changes are documentation-only and do not introduce new multi-component control flow)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • Adammatthiesen
  • dreyfus92

Poem

🐇📚 I nibbled at lines, then hopped to the trees,
I swapped some imports and pruned some MDX leaves,
The Kysely client now hums in French light,
A new SDK page sparkles, tidy and bright,
Hooray — docs refreshed, and the meadow feels right.

🚥 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 accurately describes the main change: moving and reorganizing French documentation files from utils/kysely-and-sdk.mdx to separate files in ecosystem/packages/.
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 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.

Copy link
Copy Markdown
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: 2

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/content/docs/fr/ecosystem/packages/kysely.mdx (1)

260-277: Rename the example function to match the “get user” use case.

This section is about fetching a user by ID, but the exported function is named insertUser. Rename it to avoid confusion and avoid shadowing the inner helper.

✏️ Suggested fix
-export const insertUser = Effect.gen(function* () {
+export const getUserById = Effect.gen(function* () {
     const { withCodec } = yield* getDbClient;

-    const getUserById = withCodec({
+    const fetchUserById = withCodec({
         encoder: Schema.String,
         decoder: Schema.UndefinedOr(StudioCMSUsersTable.Select),
         callbackFn: (db, id) =>
             db((client) =>
                 client.selectFrom('StudioCMSUsersTable')
                 .selectAll()
                 .where('id', '=', id)
                 .executeTakeFirst()
             ),
     });

-    const user = yield* getUserById('un-identifiant-utilisateur');
+    const user = yield* fetchUserById('un-identifiant-utilisateur');
🤖 Fix all issues with AI agents
In `@src/content/docs/fr/ecosystem/packages/kysely.mdx`:
- Line 174: The section heading "Exemple pour insérer un nouvel d'utilisateur
(`withCodec`)" has a grammar error—remove the stray "d'": update the heading to
"Exemple pour insérer un nouvel utilisateur (`withCodec`)" (or "Exemple
d'insertion d'un nouvel utilisateur (`withCodec`)" if you prefer an alternate
phrasing) so the French is correct; modify the heading text in the file
src/content/docs/fr/ecosystem/packages/kysely.mdx accordingly.

In `@src/content/docs/fr/ecosystem/packages/sdk.mdx`:
- Line 21: Replace the straight ASCII apostrophe in the French phrase "qu'avec"
with the typographic right single quotation mark (’), i.e., change "qu'avec" to
"qu’avec" in the string shown in the diff so the document uses correct French
typography.

Comment thread src/content/docs/fr/ecosystem/packages/kysely.mdx Outdated
Comment thread src/content/docs/fr/ecosystem/packages/sdk.mdx Outdated
ArmandPhilippot and others added 2 commits January 24, 2026 14:05
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@dreyfus92 dreyfus92 merged commit 11f267f into withstudiocms:main Jan 24, 2026
9 checks passed
@ArmandPhilippot ArmandPhilippot deleted the i18n/fr-move-kysely-sdk branch January 24, 2026 20:18
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