Skip to content

Conversation

mr-vara
Copy link
Contributor

@mr-vara mr-vara commented May 23, 2025

Proposed changes

This PR sets up the repository to use a centralized, reusable GitHub Actions workflow that automates localization file translation using Azure AI Translation (Microsoft Translator).

What’s Included

  • Calls the reusable translation workflow from the central repository.
  • Supports translation of .po and .json files with optional context stripping.
  • Allows configuration of text_domain, defaulting to the repository name if not provided.

Configuration Notes

  • Triggers on push to main and via manual workflow_dispatch.
  • Requires the following secrets:
    • TRANSLATOR_API_KEY (Azure AI Translation)
    • NEWFOLD_ACCESS_TOKEN (for creating pull requests)

This setup helps automate translation updates across repositories using a single, centralized translation logic.

Documentation

AI Translation Workflow Documentation

@circlecube
Copy link
Member

I don't see the TRANSLATOR_API_KEY secret in the repo or the org. Has it been added anywhere, @mr-vara?

@mr-vara
Copy link
Contributor Author

mr-vara commented May 29, 2025

I don't see the TRANSLATOR_API_KEY secret in the repo or the org. Has it been added anywhere, @mr-vara?

@circlecube No, it has to be added on org level and I don't have permissions for the same. I have the key with me, but need help with adding it to org level.

mr-vara and others added 3 commits June 10, 2025 11:28
Co-authored-by: Jonathan Desrosiers <359867+desrosj@users.noreply.github.com>
Co-authored-by: Jonathan Desrosiers <359867+desrosj@users.noreply.github.com>
@mr-vara mr-vara requested review from desrosj and alarocca130 June 10, 2025 14:43
Copy link
Member

@desrosj desrosj left a comment

Choose a reason for hiding this comment

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

Looks good, @mr-vara! Just one thing missing here. Adding concurrency will ensure that multiple workflows are not trying to create pull requests for the same branch at the same time.

Co-authored-by: Jonathan Desrosiers <359867+desrosj@users.noreply.github.com>
@mr-vara mr-vara requested a review from desrosj June 20, 2025 04:55
@mr-vara mr-vara merged commit 917beae into main Jun 23, 2025
9 of 19 checks passed
@mr-vara mr-vara deleted the PRESS11-157 branch June 23, 2025 05:57
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.

5 participants