Skip to content

i18n - translations#17025

Merged
FelixMalfait merged 2 commits intomainfrom
i18n
Jan 8, 2026
Merged

i18n - translations#17025
FelixMalfait merged 2 commits intomainfrom
i18n

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Jan 8, 2026

Created by Github action

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.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 62 files

@FelixMalfait FelixMalfait merged commit 85a7f65 into main Jan 8, 2026
10 checks passed
@FelixMalfait FelixMalfait deleted the i18n branch January 8, 2026 16:27
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions bot commented Jan 8, 2026

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-00ca5868.json

Generated by 🚫 dangerJS against 65ef7f1

FelixMalfait added a commit that referenced this pull request Mar 25, 2026
PR #16997 replaced decorator-based workspace entities (which used
msg`` for labels) with flat metadata builders using plain strings.
The next automated lingui extract (PR #17025, 42 minutes later)
removed all standard object label entries from the PO files, erasing
their French translations.

Restores 32 translations (Company→Entreprise, Person→Personne, etc.)
from the state before erasure. Rewrites the integration test to verify
actual French translations via the metadata API.

Made-with: Cursor
FelixMalfait added a commit that referenced this pull request Mar 25, 2026
Restore 319 French translations that were erased in PR #17025 when
lingui extract ran after PR #16997 removed msg`` from workspace
entities. This includes object labels, field labels, and descriptions.

Fix i18n integration test to update workspace member locale (the
x-locale header is ignored for authenticated requests since
userWorkspace.locale takes precedence).

Made-with: Cursor
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