Skip to content
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

addresses from addressbook's sending tab appear in listaccounts #76

Closed
tcatm opened this issue Feb 22, 2011 · 5 comments
Closed

addresses from addressbook's sending tab appear in listaccounts #76

tcatm opened this issue Feb 22, 2011 · 5 comments
Labels

Comments

@tcatm
Copy link

tcatm commented Feb 22, 2011

Addresses from the sending tab in wxGUI's addressbook appear in listaccounts. The RPC will happily return that address even though it isn't yours.

This is a problem when switching from wxGUI to a RPC UI.

@lachesis
Copy link
Contributor

Submitted pull request: #126

@TheBlueMatt
Copy link
Contributor

Aside from the typo in the code, is there a reason why this pull request was closed?

@lachesis
Copy link
Contributor

lachesis commented May 8, 2011

No. I submitted a second pull request (#127,
#127) with revised code, which
was merged on April 5th.

On 05/07/2011 08:11 PM, TheBlueMatt wrote:

Aside from the typo in the code, is there a reason why this pull request was closed?

Eric Swanson
http://www.alloscomp.com/
GPG Key: 7A96 A45E 1603 6FCB EA72 146D 33D6 60E9 FFC7 420D

@TheBlueMatt
Copy link
Contributor

Can we close this issue then?

@lachesis
Copy link
Contributor

lachesis commented May 8, 2011

Yeah, the bug is fixed in the current HEAD, so close it.

On 05/07/2011 08:15 PM, TheBlueMatt wrote:

Can we close this issue then?

Eric Swanson
http://www.alloscomp.com/
GPG Key: 7A96 A45E 1603 6FCB EA72 146D 33D6 60E9 FFC7 420D

@sipa sipa closed this as completed May 8, 2011
glv2 referenced this issue in glv2/peercoin Jul 22, 2014
Update of Norwegian (Bokmål) translation
rdponticelli pushed a commit to Criptomonedas/bitcoin that referenced this issue Nov 26, 2014
7935930 Add unit tests for scalars. (Pieter Wuille)
eca6cdb Switch scalar to use get/set 32-byte arrays (Pieter Wuille)
kac- pushed a commit to kac-/peercoin-ce that referenced this issue Jan 23, 2015
jtimon pushed a commit to jtimon/bitcoin that referenced this issue Mar 11, 2016
5f5ebe2 Re-organize the flow of the README, include Quick Start. (Eric Martindale)
932e66b Add and improve verbiage and messaging. (Eric Martindale)
a681cb7 Add links to dedicated pages describing each Element. (Eric Martindale)
destenson pushed a commit to destenson/bitcoin--bitcoin that referenced this issue Jun 26, 2016
Fixed lookup for correct vin in GetMasterNodeVin when txHash is availabl...
classesjack pushed a commit to classesjack/bitcoin that referenced this issue Jan 2, 2018
lateminer pushed a commit to lateminer/bitcoin that referenced this issue Feb 23, 2019
cryptapus pushed a commit to cryptapus/bitcoin that referenced this issue May 3, 2021
Allow setting the nameop argument to null for createrawtransaction while
still setting the locktime.  This fixes bitcoin#76.
rajarshimaitra pushed a commit to rajarshimaitra/bitcoin that referenced this issue Aug 5, 2021
glossary.asciidoc: add Bech32 description
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants