Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

WIP: add script to build and upload crowdin glossary #86

Merged
merged 1 commit into from
May 17, 2018

Conversation

lukaszewczak
Copy link
Contributor

Hi,

So according to #81 I add this simple script to build and upload crowdin glossary. I put this script in WIP status due to today's meeting and because I do not know how to test it correctly, I do not have crowdin key, and I want to write unit tests for it.

@lukaszewczak lukaszewczak requested review from zeke and maddhruv May 15, 2018 12:09
zeke

This comment was marked as off-topic.

@zeke
Copy link
Contributor

zeke commented May 15, 2018

I have confirmed that this is unchecked in Crowdin:

screen shot 2018-05-15 at 1 13 49 pm

This prevents surprises by ensuring that translators won't create entries that will be removed by our automated Glossary updated.

IMO, this PR can be merged at any time. 👍

maddhruv

This comment was marked as off-topic.

maddhruv

This comment was marked as off-topic.

@maddhruv maddhruv merged commit e89bd58 into master May 17, 2018
@maddhruv maddhruv deleted the upload-crowdin-glossary branch May 17, 2018 12:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants