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

[Smart Contract Lang Page] Remove Turkish Docs Link in Solidity Section #2969

Closed
wants to merge 1 commit into from

Conversation

franzihei
Copy link
Member

@franzihei franzihei commented May 4, 2021

Hi there!

I propose removing the link to the "Turkish Solidity Documentation".

A few reasons and some context:

  • There are plenty of efforts for community translations of the Solidity documentation (e.g. in French, Korean, Chinese, Italian, Spanish, Japanese, and more), not only the Turkish one. Some of them are in a more up-to-date state and some of them are in a somewhat unfinished / out-of-date state. (See https://docs.soliditylang.org/en/latest/index.html#translations.)
  • Linking to unfinished or out-of-date translations in this section is not helpful.
  • We recently set up a new translation process which should help streamline the translation efforts and give translations the chance to be featured in the "official" Solidity documentation once they reach a certain state of completeness and quality.
  • The process and all details about it can be found here [DOCS] Translation Workflow for Docs solidity#10119 (comment) and here https://github.com/solidity-docs/translation-guide.
  • The Turkish community translation does not follow any of these guidelines, is nowhere near a complete translation, does not include versioning (or any info on from which date this translation is) and can clearly not be seen as an "important link" (I personally think none of the translations should be featured there, it would crowd the link section too much).

In general, the Solidity team would highly appreciate tagging one of us in the review before merging stuff into the Solidity section! In most cases, we do have a bit more of context on things that can help make a reasonable decision. Thanks :)

@ryancreatescopy
Copy link
Contributor

Looks like this link was removed as part of #2970 too, so might just be worth closing this one @wackerow

@wackerow
Copy link
Member

wackerow commented May 6, 2021

@franzihei thanks for keeping an eye on, appreciate the info =)...
Will close in lieu of #2970

@wackerow wackerow closed this May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants