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

Update texts about multi-signature #637

Merged
merged 2 commits into from Nov 9, 2014

Conversation

Projects
None yet
2 participants
Contributor

saivann commented Nov 8, 2014

Live preview: (Merged)

Some texts about multisig are a little outdated and inaccurate now (describe it as a soon-to-become-usable feature). I've tried to refresh these texts, and took the opportunity of improving them with more concise and relevant examples.

@harding harding commented on an outdated diff Nov 8, 2014

_translations/en.yml
@@ -36,7 +36,7 @@ en:
visibility: "Get some free visibility"
visibilitytext: "Bitcoin is an emerging market of new customers who are searching for ways to spend their bitcoins. Accepting them is a good way to get new customers and give your business some new visibility. Accepting a new payment method has often shown to be a clever practice for online businesses."
multisig: "Multi-signature"
- multisigtext: "Bitcoin also includes a feature, not yet well known, which allows bitcoins to be spent only if a subset of a group of people sign the transaction (so-called \"m of n\" transactions). This is the equivalent of the good old multi-signature cheque system that you might still use with banks today."
+ multisigtext: "Bitcoin also includes a multi-signature feature which allows bitcoins to be spent only if a subset of a group of people authorize the transaction. This can be used by a board of directors to prevent any member to engage expenditures without enough consent from other members, as well as to track which members allowed each payment."
@harding

harding Nov 8, 2014

Contributor

Strong recommendation: s/engage/make/

@harding harding commented on an outdated diff Nov 8, 2014

_translations/en.yml
@@ -546,7 +546,7 @@ en:
update: "Keep your software up to date"
updatetxt: "Using the latest version of your Bitcoin software allows you to receive important stability and security fixes. Updates can prevent problems of various severity, include new useful features and help keep your wallet safe. Installing updates for all other software on your computer or mobile is also important to keep your wallet environment safer."
offlinemulti: "Multi-signature to protect against theft"
- offlinemultitxt: "Bitcoin includes a multi-signature feature that allows a transaction to require the signature of more than one private key to be spent. It is currently only usable for technical users but a greater availability for this feature can be expected in the future. Multi-signature can, for example, allow an organization to give access to its treasury to its members while only allowing a withdrawal if 3 of 5 members sign the transaction. It can also allow future online wallets to share a multi-signature address with their users, so that a thief would need to compromise both your computer and the online wallet servers in order to steal your funds."
+ offlinemultitxt: "Bitcoin includes a multi-signature feature that allows a transaction to require multiple independent approvals to be spent. This can be used by an organization to give its members access to its treasury while only allowing a withdrawal if 3 of 5 members sign the transaction. Some web wallets also provide multi-signature wallets, allowing the user to keep control over their money while preventing a thief to steal funds by compromising a single device or server."
@harding

harding Nov 8, 2014

Contributor

In "a thief to steal funds", s/to/from/

Edit: whoops: should be, "prevent a thief from stealing funds"

Contributor

saivann commented Nov 8, 2014

@harding Thanks, your help is always appreciated!

Contributor

saivann commented Nov 8, 2014

In the absence of critical feedback. this pull request will be merged on November 9th.

saivann added a commit that referenced this pull request Nov 9, 2014

Merge pull request #637 from bitcoin/multisigrefresh
Update texts about multi-signature

@saivann saivann merged commit cca9956 into master Nov 9, 2014

@saivann saivann deleted the multisigrefresh branch Nov 9, 2014

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