Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add "New app" as a "Transparency" score for new wallets with a short track record #503

Merged
merged 2 commits into from Aug 8, 2014

Conversation

Projects
None yet
2 participants
Contributor

saivann commented Aug 4, 2014

There seems to be a few interesting wallets (Aegis, Breadwallet, Ciphrex) which may be good choices for the user. However, many of them have been released only more recently.

This pull request adds a "Transparency" score text which warns the user that such wallets still don't have a very long track record, in order to find a good compromise that would allow us to list them while being transparent about the few public testing / review these wallets may have received yet.

Contributor

harding commented Aug 4, 2014

I like this idea. I suggest we change the second sentence from:

However, this wallet still has a short track record and few public feedback, testing and reviews.

To:

However, this wallet has not been tested and publicly reviewed by a significant number of people.

This removes the metaphor/colloquialism "short track record" which may not be clear to certain non-native English speakers and so should allow for a smoother translation.

It also removes the time dependency here: in my opinion, it's the lack of reviews, not the age of the wallet, that matters most---so by not mentioning time, we can now apply this warning even to older wallets. (I think one of the wallets mentioned in the OP said it was over a year old---yet I had never heard of it.)

Contributor

saivann commented Aug 4, 2014

@harding Thanks! I think your version is indeed much better.

Contributor

saivann commented Aug 7, 2014

In the absence of critical feedback, this pull request will be merged on August 8th.

saivann added a commit that referenced this pull request Aug 8, 2014

Merge pull request #503 from bitcoin/checkfailtransparencynew
Add "New app" as a "Transparency" score for new wallets with a short track record

@saivann saivann merged commit bdcb8be into master Aug 8, 2014

@saivann saivann deleted the checkfailtransparencynew branch Aug 8, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment