Skip to content

i18n - translations#16227

Merged
FelixMalfait merged 2 commits intomainfrom
i18n
Dec 1, 2025
Merged

i18n - translations#16227
FelixMalfait merged 2 commits intomainfrom
i18n

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Dec 1, 2025

Created by Github action

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Dec 1, 2025

Greptile Overview

Greptile Summary

Automated translation extraction PR that adds three new translatable strings for chart configuration settings.

  • Added Prefix, Suffix, and unit translation keys to English locale (en.po) with proper values
  • Added placeholder entries (empty msgstr) for all non-English locales (31 languages) awaiting community translation
  • Regenerated all TypeScript locale files (generated/*.ts) with compiled translations
  • Changes follow the standard i18n workflow: extract translations → compile → generate TypeScript files

All changes appear to be automatically generated by the github-actions bot following the project's translation extraction process. The new strings originate from chart configuration settings (ChartConfigurationSettingLabels.ts, PrefixSetting.ts, SuffixSetting.ts).

Confidence Score: 5/5

  • This PR is safe to merge - it only contains automated translation extraction changes with no code logic modifications
  • Perfect score reflects that this is a routine automated translation update with no functional changes, consistent formatting, and proper i18n structure following established patterns
  • No files require special attention - all changes follow automated translation extraction process

Important Files Changed

File Analysis

Filename Score Overview
packages/twenty-front/src/locales/en.po 5/5 Added three new English translation entries: Prefix, Suffix, and unit for chart configuration settings
packages/twenty-front/src/locales/generated/en.ts 5/5 Compiled TypeScript translation file updated with new translation entries from en.po
packages/twenty-front/src/locales/fr-FR.po 4/5 Added placeholder entries for Prefix, Suffix, and unit (empty msgstr values awaiting translation)
packages/twenty-front/src/locales/de-DE.po 4/5 Added placeholder entries for Prefix, Suffix, and unit (empty msgstr values awaiting translation)
packages/twenty-front/src/locales/generated/fr-FR.ts 5/5 Compiled TypeScript translation file updated for French locale
packages/twenty-front/src/locales/es-ES.po 4/5 Added placeholder entries for new translation strings (empty msgstr values awaiting translation)

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Code as Source Code
    participant Extract as Translation Extraction
    participant PO as .po Files
    participant Compile as Compilation
    participant TS as Generated .ts Files
    participant GH as GitHub Actions

    Dev->>Code: Add new msg`Prefix`, msg`Suffix`, msg`unit`
    Dev->>Code: Commit to ChartConfigurationSettingLabels.ts
    Code->>GH: Trigger translation workflow
    GH->>Extract: Run extract translations script
    Extract->>PO: Extract new strings to en.po (with values)
    Extract->>PO: Add placeholders to 31 other locales (empty msgstr)
    Extract->>GH: Commit extracted .po files
    GH->>Compile: Run compile translations script
    Compile->>TS: Generate TypeScript from .po files
    Compile->>TS: Update all generated/*.ts files
    Compile->>GH: Commit compiled .ts files
    GH->>GH: Create PR #16227
Loading

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

62 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@FelixMalfait FelixMalfait merged commit da75361 into main Dec 1, 2025
8 checks passed
@FelixMalfait FelixMalfait deleted the i18n branch December 1, 2025 18:01
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions bot commented Dec 1, 2025

Fails
🚫

node failed.

Log

Details
�[31mError: �[39m SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
�[90m    at parseJSONFromBytes (node:internal/deps/undici/undici:6433:19)�[39m
�[90m    at successSteps (node:internal/deps/undici/undici:6414:27)�[39m
�[90m    at readAllBytes (node:internal/deps/undici/undici:5380:13)�[39m
�[90m    at process.processTicksAndRejections (node:internal/process/task_queues:103:5)�[39m
danger-results://tmp/danger-results-127bb986.json

Generated by 🚫 dangerJS against e96b1d9

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