Skip to content
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

Add crowdin action #2020

Merged
merged 5 commits into from
Oct 9, 2024
Merged

Add crowdin action #2020

merged 5 commits into from
Oct 9, 2024

Conversation

anbraten
Copy link
Contributor

With crowdin/crowdin-cli#776 and crowdin/crowdin-cli#794 deleting obsolete files which got removed in the source language (en) works now using the cli which is used by this github action. Therefore this PR introduces the Crowdin action to replace the old integration.

Steps needed before merging

  • disable the integration in crowdin
  • generate an api token in crowdin and save it as secret CROWDIN_PERSONAL_TOKEN in github

@anbraten
Copy link
Contributor Author

@MilosKozak Afaik as you are the repo and crowdin project owner, you would need to support us with the secrets. (hope this is correct and sorry for pinging)

CC @peterleimbach

@MilosKozak
Copy link
Collaborator

I can do it. Just tell me what you need

@anbraten
Copy link
Contributor Author

  1. Disable the github integration in crowdin: https://crowdin.com/project/androidapsdocs/integrations/system/github
    Should be enough if you click "pause sync"
  2. Create a new crowdin api key: https://crowdin.com/settings#api-key and save it in github https://github.com/anbraten/repro-crowdin-delete-obsolete/settings/secrets/actions as secret: CROWDIN_PERSONAL_TOKEN

Thanks

@peterleimbach
Copy link
Collaborator

@Milos: Could you implement @anbraten requirements for the actions in Github? Can merge this PR?

@psonnera psonnera merged commit 4cce5f9 into openaps:master Oct 9, 2024
2 checks passed
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.

4 participants