Skip to content

add turkish wordlist#1359

Closed
hknylcnsy wants to merge 1 commit intobitcoin:masterfrom
hknylcnsy:master
Closed

add turkish wordlist#1359
hknylcnsy wants to merge 1 commit intobitcoin:masterfrom
hknylcnsy:master

Conversation

@hknylcnsy
Copy link
Copy Markdown

  1. Simple and common Turkish words.
  2. Only words with 4-8 letters.
  3. The first 4 letters are unique among all words.
  4. The letters (â,î,û,ô) used in Turkish are considered equal to (a,e,u/ü,o/ö). (like: kâğıt-kağıt, sükût-sükut)
  5. No complex verb forms.
  6. No plural words.
  7. No offensive/negative words.
  8. No words with double vowels. (like: maalesef, şuur)
  9. No words to mingle. (like: adam-aday, alet-alev, surat-suret)
  10. No words with repeated syllables. (like: leblebi, baba)
  11. No infinitive words. (like: yapmak, gülmek)

@izzetemredemir
Copy link
Copy Markdown

+1 Well prepared

@luke-jr
Copy link
Copy Markdown
Member

luke-jr commented Sep 29, 2022

Copy link
Copy Markdown

@zinderud zinderud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/zinderud/scure-bip39
I changed the letters in some words in Turkish (ü,ğ,ş,ı,ö) to their English equivalents (u,g,s,i,o) to be compatible with the tests.

// Normalization replaces equivalent sequences of characters
// so that any two texts that are equivalent will be reduced
// to the same sequence of code points, called the normal form of the original text.

https://tonsky.me/blog/unicode/#why-is-a----

@zinderud
Copy link
Copy Markdown

@murchandamus
Copy link
Copy Markdown
Member

murchandamus commented Apr 26, 2024

If BIP39 no longer accepts new wordlists, and such pull requests should instead be directed at the WLIPs repository, it seems to me that this PR can be closed.

I propose to close this PR after 2024-05-10 unless further activity indicates otherwise.

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.

5 participants