-
Notifications
You must be signed in to change notification settings - Fork 129
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
Update StakeShuffle/Privacy Documentation #1153
base: master
Are you sure you want to change the base?
Conversation
72073c6
to
421b97e
Compare
Update verbiage on CSPP/StakeShuffle with more recent developments. Adds terms CoinShuffle++ and StakeShuffle to glossary. Adds video tutorial for privacy mixing on "How To" page. Addresses decred#1151 and decred#1152.
421b97e
to
4ec350a
Compare
Can you update the urls and provide redirects for the old urls? You could add a definition for CoinJoin to the glossary for consistency. |
Rebranding this feature is silly, and afaict, was not discussed or approved anywhere else. The protocol is still CoinShuffle++ no matter how it is used and what amounts are mixed. In fact, the server does not even know the current ticket price. It only groups peers which are mixing the same amounts. |
The function of CSPP is distinct from coinjoins on chain. Due to the original branding of CSPP, people are frequently using the term incorrectly to describe the properties of the coinjoins on chain. CSPP is the client-server protocol for creating the coinjoin transactions. StakeShuffle was proposed as the branding for what people have been calling Decred Privacy. It makes a lot of sense since the primary usage is with the ticket system. The goal is to make this easier to understand for users and for terms to be used correctly. |
I had some reservations about these changes myself that I'm more concerned with after jrick's response. CSPP is referenced a lot in the underlying code and documentation for the applications in their current versions. A rebranding needs wider consensus between developers and marketing. I think it may be best to revert most of the rebranding changes and perhaps just include a note that CSPP CoinJoin transactions used in combination with Staking have been referred to as "StakeShuffle". Calling it all StakeShuffle when it's not only ticket purchasing transactions that are mixed definitely seems off. This change introduces more confusion in it's current state. I will clean up and separate my changes into separate commits. |
Agreed @dezryth, if there is not enough consensus we need to discuss this further and undo the relevant changes here to not block this PR. |
There has been more discussion on this topic and I feel more comfortable that this PR addresses the current takeaways I saw in this discussion.
I think my PR here accomplishes this actually, and I’m seeing more support than I expected for the branded name for the mixing process than I initially thought existed between marketing/dev. Can someone take a look at the changes here and let me know what parts in particular you take issue with in the review? I want to move it forward and this most recent conversation has been really helpful. |
Great work! Looks good to me. I thought it might be valuable to link to the new glossary terms in the "Overview" section, but then I realized you had pretty much the exact same content covered there, so it would be redundant. |
I'd also replace the |
This PR addresses #1151 and #1152.
Please review and let me know of any changes anyone would like to see before merging. Given this is a very important area to Decred stakeholders, I want to ensure the verbiage changes are satisfactory.