Skip to content

Conversation

@comfyui-wiki
Copy link
Member

@comfyui-wiki comfyui-wiki commented Jul 9, 2025

Update all the template translations.

┆Issue is synchronized with this Notion page by Unito

@christian-byrne
Copy link
Contributor

christian-byrne commented Jul 9, 2025

@comfyui-wiki In the code, we normalize the translation keys. That includes converting dots to underscores. So, if any template filename has dots in it, we have to convert to underscores when adding them to the translation files. I should have mentioned that in the README.md of Comfy-Org/workflow_templates.

comfyui-wiki and others added 6 commits July 10, 2025 15:23
- Accept HEAD version (720e0a9) for all locale files
- Restore Comfy_Notification_ShowVersionUpdates setting in settings.json
- Ignore GitHub Action's alphabetical reorganization
- Keep original template structure intact
@comfyui-wiki
Copy link
Member Author

Just about to add the zh-TW translation of templates and found that the GitHub Action had already finished it. 👍

@comfyui-wiki
Copy link
Member Author

image image image All the template translations should now display normally.

Copy link
Contributor

@christian-byrne christian-byrne left a comment

Choose a reason for hiding this comment

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

So far I have been using "LoRA" instead of "Lora". I think "LoRA" is the standard ML terminology (Low Rank Adaption ...). WDYT?

@comfyui-wiki
Copy link
Member Author

I see why the check failed now. The original default template's description was inconsistent with others, so I updated the text.

image image image

Both failed checks are for the same reason.

@christian-byrne christian-byrne added the New Browser Test Expectations New browser test screenshot should be set by github action label Jul 11, 2025
Copy link
Contributor

@christian-byrne christian-byrne left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@christian-byrne christian-byrne merged commit 80e5cf1 into main Jul 11, 2025
2 checks passed
@christian-byrne christian-byrne deleted the update-the-templates-translation branch July 11, 2025 06:10
@christian-byrne christian-byrne mentioned this pull request Jul 11, 2025
3 tasks
jtydhr88 pushed a commit that referenced this pull request Jul 12, 2025
benceruleanlu pushed a commit that referenced this pull request Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New Browser Test Expectations New browser test screenshot should be set by github action

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants