Skip to content

i18n(de): start German translation; key-values translated#46

Merged
louisescher merged 1 commit into
withstudiocms:mainfrom
trueberryless:de
Apr 11, 2025
Merged

i18n(de): start German translation; key-values translated#46
louisescher merged 1 commit into
withstudiocms:mainfrom
trueberryless:de

Conversation

@trueberryless
Copy link
Copy Markdown
Member

@trueberryless trueberryless commented Apr 11, 2025

Description

This PR decomments German configs and adds all json files in German.
If I wanna preview in dev mode or build the project, I get an error tho, not sure why the starlight-sidebar/en.json file doesnt exist, not related tho...

Pretty made more changes to starlight-sidebar/translate.ts than necessary.

Summary by CodeRabbit

  • New Features
    • Enabled German language support throughout the application.
    • Introduced comprehensive German translations for interface elements, including site labels, navigation cues, and user messages, ensuring a consistent and localized user experience.

@trueberryless trueberryless requested a review from a team as a code owner April 11, 2025 17:05
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 11, 2025

Walkthrough

The pull request adds support for the German locale. It updates the configuration in the primary settings file to enable German, introduces two new JSON files containing German translations for the main content and sidebar, and expands the list of supported languages in the translation module. These changes ensure that the application can now load and display German-language text for its user interface without altering any public API or core functionality.

Changes

File(s) Change Summary
astro.config.mts Updated the locales configuration by uncommenting and enabling the German language entry.
src/content/i18n/de.json
starlight-sidebar/de.json
Introduced new JSON files with key-value pairs for German translations covering UI labels, messages, and sidebar topics across the application.
starlight-sidebar/translate.ts Updated the translations array to include "de", reformatted the import statement to use double quotes, and adjusted indentation from tabs to spaces.

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant Config
  participant TranslationService
  participant UI

  User->>Config: Request available locales
  Config-->>User: Return locales with German enabled
  User->>TranslationService: Select language "de"
  TranslationService->>src/content/i18n/de.json: Load UI translations
  TranslationService->>starlight-sidebar/de.json: Load sidebar translations
  TranslationService-->>User: Return German translation data
  User->>UI: Render interface with German translations
Loading

Tip

⚡💬 Agentic Chat (Pro Plan, General Availability)
  • We're introducing multi-step agentic chat in review comments and issue comments, within and outside of PR's. This feature enhances review and issue discussions with the CodeRabbit agentic chat by enabling advanced interactions, including the ability to create pull requests directly from comments and add commits to existing pull requests.
✨ Finishing Touches
  • 📝 Generate Docstrings

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai plan to trigger planning for file edits and PR creation.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@studiocms-no-reply
Copy link
Copy Markdown
Member

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
src/content/i18n/de.json Localization added, will be marked as complete.
starlight-sidebar/de.json 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.

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: 0

🧹 Nitpick comments (1)
starlight-sidebar/translate.ts (1)

5-5: Consider reverting formatting changes that don't affect functionality.

As mentioned in the PR description, more modifications were made to this file than necessary. The changes from single quotes to double quotes and from tabs to spaces don't affect functionality but might not align with the project's coding style.

-import fallback from "./en.json";
+import fallback from './en.json';

-const Translations = ["en", "fr", "ko", "de"] as const;
+const Translations = ['en', 'fr', 'ko', 'de'] as const;

-  Translations.map(async (t) => {
-    return { lang: t, data: (await import(/* @vite-ignore */ `./${t}.json`)).default };
-  })
+	Translations.map(async (t) => {
+		return { lang: t, data: (await import(/* @vite-ignore */ `./${t}.json`)).default };
+	})

// Similar changes for other indentation changes in the file

Also applies to: 12-14, 18-30

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 807b02e and b41619e.

📒 Files selected for processing (4)
  • astro.config.mts (1 hunks)
  • src/content/i18n/de.json (1 hunks)
  • starlight-sidebar/de.json (1 hunks)
  • starlight-sidebar/translate.ts (1 hunks)
🔇 Additional comments (4)
astro.config.mts (1)

19-19: LGTM! Successfully enabled German locale.

The German locale has been properly uncommented in the configuration file, which aligns with the PR objective of initiating German translation support.

starlight-sidebar/translate.ts (1)

7-7: LGTM! German language correctly added to the translations array.

The German locale "de" has been successfully added to the supported translations list, which is necessary for enabling German language support.

starlight-sidebar/de.json (1)

1-16: LGTM! German sidebar translations look good.

The German translations for the sidebar items are accurate and properly formatted. This file is essential for providing the German sidebar navigation experience.

src/content/i18n/de.json (1)

1-48: LGTM! Comprehensive German translations for site content.

The German translations cover all necessary UI elements, including site titles, documentation references, search functionality, and error messages. The translations appear accurate and well-formatted.

@louisescher louisescher merged commit 36b359b into withstudiocms:main Apr 11, 2025
6 of 7 checks passed
@trueberryless trueberryless deleted the de branch April 12, 2025 04:41
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