-
Notifications
You must be signed in to change notification settings - Fork 36.6k
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
Bitcoin-Qt signmessage GUI #582
Conversation
Does this functionality also allow for verifying a signed message? (I checked quickly and was unable to find it) |
No, it doesn't. This gets users able to work with webservices. The next logical steps once this is merged would be 1) verify, and/or 2) sign & email from the Send page |
I'm not entirely convinced that signing/verifying messages should be this prominent in the bitcoin UI. I'm fine with adding it in the menu, but is it central enough to the workflow to warrant adding a tab for it? I tried to keep it as simple as possible with the tabs. What does the rest think? |
I would agree with laanwj here, a tab should (IMHO) be reserved for something which an average person would do on a regular basis, which message signing just isnt. |
So what happens when you open the signing panel? just have none of the tabs selected? |
New window? |
I kindof like Bitcoin-Qt's single-window design. |
I agree that this is a dilemma. It would be nice to fit it in the single-window design, but without exposing it as a first-class operation. Making the option selectable from the menu bar, then showing it in the main pane while deselecting all of the tabs seems fine with me. This gives it a bit of a 'hidden feature' feel. But as there is no verify functionality yet, it still feels pretty experimental anyway. |
Ok, I made it only on the File menu, but still appears in the single-window style (deselecting the other tabs). |
I would argue that putting it in the main window but without selecting any tabs is worse than having it in either a new window or a tab. The 'hidden feature' feel never sits well with me when I see it in an application - it just makes me feel like no effort went into UI design and it was just slapped in there, which is never a good feeling to have, especially in financial software where every effort needs to be made to encourage trust in the application (ha like we do that now...) |
You have a point, but in that case it would be better to not include it all Still I like experimental features and playing with them. Sometimes you So that's my reasoning to pull this and somehow make it hidden for "normal On Thu, Oct 13, 2011 at 11:31 PM, Matt Corallo <
|
This feature is finished, as soon as someone decides whether it's allowed to have a tab or not. It provides useful functionality that people need /yesterday/. It's not some "hey, this might be useful for new stuff", it's "hey, people are already needing to use this even before it's got a final release". |
Well the consensus seems to be a separate window for now. But please do explain to us (or point to the wiki) how this is useful in
|
This is a very useful addition for many merchants or pools who want to verify a user based on bitcoin address. Although I would strongly argue for merchants doing their own auth instead of wasting time with complicated sigh-message stuff, hence why I would argue for this not taking a very central role in the gui. (also, luke stop talking about your need for things in the third person) "Wladimir J. van der Laan" reply@reply.github.com wrote:
|
In most cases, this enables merchants to just publish a single Bitcoin address for payments on their website, and if there is any confusion over who paid request individual customers sign their receipt with a sending address. It is also useful, as BlueMatt implied, for proving an Eligius miner controls an account (which is identified only by its payout address). Both of these problems have been around for a while now. ThomasV also mentioned that he has been waiting for this feature for some unspecified purpose. |
ACK on the code changes for release-after-0.5. I don't have an opinion on file menu versus tab. |
First time I look at signmessage_gui:
|
ACK |
Has been merged |
Is there any documentation anywhere explaining what this feature is for and when it should be used? I suspect most bitcoin users won't know what to use this for otherwise. |
Please don't comment on long-closed issues. Open a new one. |
Made one sentence out of these two.
No description provided.