Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Nov 3, 2025

Created by Github action

Copy link
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.

Greptile Overview

Greptile Summary

This PR is an automated i18n translation update generated by GitHub Actions. The changes include:

  • New translation strings added: Approximately 10 new translation keys related to the recently added SSO bypass feature (PR #15417), including:

    • Bypass SSO button text in footer
    • Allow email & password login for SSO bypass users
    • Allow Google-based login for users with SSO bypass permissions
    • Allow Microsoft-based login for users with SSO bypass permissions
    • Configure fallback login methods for users with SSO bypass permissions
    • Enable bypass options
    • Invalid auth bypass provider error message
  • Translation scope: Updates span across 93 locale files:

    • 31 email locale files (packages/twenty-emails/src/locales/*.po)
    • 31 frontend locale files (packages/twenty-front/src/locales/*.po)
    • 31 generated TypeScript files (packages/twenty-front/src/locales/generated/*.ts)
  • Translation status: English base translations are complete. Most other languages have empty msgstr fields (awaiting translation from Crowdin or translators)

  • Email locale files: Some reorganization/reordering of existing entries (no functional changes)

  • Generated files: TypeScript compilation files updated to include new translation keys in the minified JSON format

Confidence Score: 5/5

  • This PR is safe to merge with no risk - it's an automated translation update from a trusted GitHub Actions workflow
  • Score reflects that this is a standard, automated i18n translation extraction and compilation process with no logic changes. The PR only adds new translation strings for the SSO bypass feature and regenerates compiled translation files. All changes are mechanical and follow the established translation workflow pattern.
  • No files require special attention. All changes are standard i18n updates generated by the automated translation workflow.

Important Files Changed

File Analysis

Filename Score Overview
packages/twenty-front/src/locales/en.po 5/5 Added new SSO bypass-related translations for auth settings and footer components
packages/twenty-front/src/locales/generated/en.ts 5/5 Compiled translation file with new SSO bypass strings added to the JSON structure
packages/twenty-emails/src/locales/en.po 5/5 Reordered translation entries (no content changes, just reorganization)
packages/twenty-front/src/locales/fr-FR.po 5/5 Added new SSO bypass translations with empty msgstr fields awaiting translation

Sequence Diagram

sequenceDiagram
    participant GHA as GitHub Actions
    participant Src as Source Code
    participant Extract as Translation Extractor
    participant PO as PO Files
    participant Compile as Compiler
    participant TS as Generated TS Files

    Note over GHA,TS: Automated i18n Translation Workflow
    
    GHA->>Src: Detect new/changed code with Trans/t`` tags
    Src->>Extract: Run extract translations command
    Extract->>PO: Update .po files with new msgid entries
    Note over PO: Base language (en.po) has msgstr filled
    Note over PO: Other languages have empty msgstr
    
    PO->>Compile: Run compile translations command
    Compile->>TS: Generate minified JSON in TS files
    
    TS->>GHA: Commit all changes
    GHA->>GHA: Create Pull Request
Loading

93 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@FelixMalfait FelixMalfait merged commit 3dac899 into main Nov 3, 2025
6 checks passed
@FelixMalfait FelixMalfait deleted the i18n branch November 3, 2025 10:48
@github-actions
Copy link
Contributor Author

github-actions bot commented Nov 3, 2025

Fails
🚫

node failed.

Log

�[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:105:5)�[39m
danger-results://tmp/danger-results-3615ac92.json

Generated by 🚫 dangerJS against a7eeba1

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.

2 participants