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

Better crowdin command documentation. #90

Closed
rickyvetter opened this issue Sep 26, 2017 · 2 comments
Closed

Better crowdin command documentation. #90

rickyvetter opened this issue Sep 26, 2017 · 2 comments
Assignees

Comments

@rickyvetter
Copy link
Contributor

Currently in the translation documentation it's recommended that you add a line like: "crowdin-upload": "export CROWDIN_DOCUSAURUS_PROJECT_ID=$YOUR_CROWDIN_ID; export CROWDIN_DOCUSAURUS_API_KEY=$YOUR_CROWDIN_API_KEY; crowdin-cli --config ../crowdin.yaml upload sources --auto-update -b master", to your package.json. What is crowdin-cli? It doesn't work the the npm i crowdin-cli package, and the recommended crowdin cli uses the crowdin command. Is there a different package I should be looking at?

@ericnakagawa
Copy link
Contributor

Hi, this is outdated.

One way to handle this is to set an environment variable in the crowdin.yaml config file. The cli will then pull keys from ENV. This also makes it easier for circle ci integration and hides the keys.

I'll add a line about installing the cli through brew or apt-get for ubuntu.

@ericnakagawa ericnakagawa self-assigned this Oct 24, 2017
@ericnakagawa
Copy link
Contributor

I have requested that Crowdin In add a public read-only method for downloading strings so that we don't need this complex step. In meantime, I'll update the docs to add instructions for installing crowdin.

#171

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

No branches or pull requests

2 participants