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

Remove Tolk, Crowdin in use #1729

Merged
merged 1 commit into from
Jul 10, 2017
Merged

Remove Tolk, Crowdin in use #1729

merged 1 commit into from
Jul 10, 2017

Conversation

bertocq
Copy link
Collaborator

@bertocq bertocq commented Jul 8, 2017

What

  • Crowdin now hosts all consul translations publicly so anyone can contribute https://crowdin.com/project/consul/invite

  • Crowdin makes easy for non-tech people to contribute, once we have a demo site we'll be able to offer "on site" translation

  • If you still use Tolk, we recommend removing this commit when you pull from upstream and adding tolk gem to ./Gemfile_custom file to avoid future merge conflicts.

How

Deployment

As usual

Warnings

  • Include a note on next release about this change, just in case a fork uses it.

Why:

* Crowdin now hosts all consul translations publicly so anyone can contribute https://crowdin.com/project/consul/invite
* Crowdin makes easy for non-tech people to contribute, once we have a demo site we'll be able to offer "on site" translation
* If you still use Tolk, we recommend removing this commit (but adding tolk gem to ./Gemfile_custom file to avoid future merge conflicts)

How:

* Reverting changes introduced with 9610a6a
@voodoorai2000
Copy link
Member

👍

@voodoorai2000 voodoorai2000 merged commit cd91608 into master Jul 10, 2017
@voodoorai2000 voodoorai2000 deleted the chore/remove_tolk branch July 10, 2017 09:52
javierm added a commit to javierm/consul that referenced this pull request Dec 7, 2018
…ilestone_validation

Fix milestone validation
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.

None yet

2 participants