-
Notifications
You must be signed in to change notification settings - Fork 2
feat: add desktop localization codegen #26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR introduces a new Python code generator for Session Desktop localization files, replacing the previous string generation approach. The new script generates TypeScript type definitions and translations in a sparse format (only actual translations, no English duplicates).
- Adds
codegen_desktop.pyto generate TypeScript localization files with type safety - Updates GitHub Actions workflow to use the new code generator
- Changes output structure to generate files under
ts/localization/generated/
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 13 comments.
| File | Description |
|---|---|
| crowdin/codegen_desktop.py | New Python script that generates TypeScript localization files (locales.ts, english.ts, translations.ts, constants.ts) with type definitions and sparse translations |
| .github/workflows/check_for_crowdin_updates.yml | Updated workflow to call the new codegen script instead of the old string generator, removed Desktop checkout step, and updated artifact paths |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Desktop changes: session-foundation/session-desktop#1761