Skip to content

Conversation

crowning-
Copy link

Reference: https://www.dash.org/forum/threads/q3-core-team-report-call.11085/#post-106703 and https://www.dash.org/forum/threads/in-wallet-privatesend-help.11324/

Final text by ddlink7 (thanks a lot 👍 ).

Advanced PrivateSend interface with complete help text:
advanced_ps

Simple PrivateSend interface:
simple_ps

The link at the end is clickable and goes to our Wiki.

@crowning-
Copy link
Author

@schinzelh , can you have a look at the Travis build?
The reason it failed has nothing to do with my PR, and you're the only one who can add packages to this whitelist anyway :-)

@schinzelh
Copy link

WTF - whitelisted packages... this is new...

I'll consider migrating from Docker based Travis to native one..

@schinzelh
Copy link

Btw: They broke Ethereum builds as well

travis-ci/travis-ci#6822 (comment)

@UdjinM6
Copy link

UdjinM6 commented Nov 4, 2016

From the description it sounds like mixing session is using 1 input from each participant and uses 1 address only which can mislead people thinking that "oh, well I have addresses for 1000 sessions, I shouldn't really care". In reality it can use up to 9 inputs per person in one mixing session iirc + it uses addresses for collaterals + sometimes mixing fails at later stage if smth goes wrong with creation of final tx (someone might be misbehaving or disconnected for some reason e.g. closed his wallet) i.e. user would see no "Denominate" tx but some of his addresses were already consumed. I think it worth fixing it at least by changing "one of your addresses" to "few of your addresses" and mentioning that such keypool size should be enough for ~100 mixing events.

@crowning-
Copy link
Author

"Up to 9 inputs", together with "about 100 mixing events" sounds good to me.

I'll change it later today.

@crowning-
Copy link
Author

New help text:
helptext2

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK

@UdjinM6 UdjinM6 merged commit eb92933 into dashpay:v0.12.1.x Nov 5, 2016
This was referenced Nov 6, 2016
tgflynn pushed a commit to tgflynn/dash that referenced this pull request Nov 7, 2016
@crowning- crowning- deleted the UI_Enhancement branch November 9, 2016 13:56
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.

3 participants