-
Notifications
You must be signed in to change notification settings - Fork 37.7k
do not create "Your Address" account when creating a new wallet #54
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Lets pull this for the 0.3.21 release-- I want to get 0.3.20 out, and don't want to pull anything that has even a slight chance of causing problems. |
ACK |
kac-
pushed a commit
to kac-/peercoin-ce
that referenced
this pull request
Sep 28, 2014
[Refs bitcoin#53] (brossi) Update Windows build dependency on libssl1.0.1b to libssl1.0.1g (openssl CVE-2014-0160)
rdponticelli
pushed a commit
to Criptomonedas/bitcoin
that referenced
this pull request
Nov 26, 2014
1c7fa13 Add VERIFY_CHECK/DEBUG_CHECK and use CHECK macros more (Pieter Wuille)
dexX7
added a commit
to dexX7/bitcoin
that referenced
this pull request
May 25, 2015
5ff7d0c Remove obsolete writing of STO receivers to file (dexX7) 9b0e2f6 As default, set block = -1 in CMPTransaction (dexX7) 4415fc2 Pass CMPTransaction to parseTransaction by reference (dexX7) aecf5e0 Move simple send checks to the top (dexX7) 5ab54ec Add new log category "packets" (dexX7) 1af418f Move MetaDEx "MSC/TMSC on one side" check into logic function (dexX7) c5bc248 Remove redundant log entries and print to log (dexX7) a8ae896 Format CMPTransaction header (dexX7) 423726b Group and tag CMPTransaction members (dexX7) 73bfd7a Interpret transaction packets in one step (dexX7) 6e3dbde Format interpretPacket (dexX7) a5974d6 Use new methods to parse packet per transaction type (dexX7) e8b4505 Add packet interpretation methods (dexX7) c2c8aa8 Move packet parsing into interpretPacket (dexX7) cdab742 Move step 3 (sp fixed) checks into logic method (dexX7) 23588b4 Move step 3 (sp variable) checks into logic method (dexX7) 276a6b1 Move step 2 (smart property) checks into logic methods (dexX7) 0598a3a Move step 2 (value) checks into logic methods (dexX7) 89976e7 Move logic into seperated logic methods (dexX7) 9c0abc5 Prepare move of logic into seperated logic methods (dexX7) db08ed9 Reorder interpretPacket based on type (dexX7) 5d41ae8 Restrict access to parseTransaction (dexX7)
core-bitcoin
pushed a commit
to core-bitcoin/bitcoin
that referenced
this pull request
Jul 14, 2017
remove not needed constant
CryptAxe
added a commit
to CryptAxe/bitcoin
that referenced
this pull request
Nov 17, 2017
CryptAxe
added a commit
to CryptAxe/bitcoin
that referenced
this pull request
Dec 28, 2017
effectsToCause
added a commit
to vericoin/vericoin
that referenced
this pull request
Jun 22, 2018
1.6.4 security updates
kewde
pushed a commit
to kewde/bitcoin
that referenced
this pull request
Jul 22, 2019
Losangelosgenetics
pushed a commit
to Losangelosgenetics/bitcoin
that referenced
this pull request
Mar 12, 2020
Firacode works on KDE4 in some environments at least.
jonasschnelli
added a commit
that referenced
this pull request
Oct 23, 2020
6954156 qt: Fix visual quality of text in QR image (Hennadii Stepanov) 8071c75 qt, refactor: Limit scope of QPainter object (Hennadii Stepanov) Pull request description: Master (197450f):  This PR (6954156): - macOS 10.15.6  - Linux Mint 20  Fix #54 Fix #19103 --- The first commit is easy to review with [`git diff --word-diff`](bitcoin-core/gui@8071c75?w=1). ACKs for top commit: jonasschnelli: Tested ACK 6954156 - tested on macOS 10.15. Fixes the problem. Tree-SHA512: 6ecb3397d2a5094c5f00ee05fc09520751568404e000a8691b6de7e57f38c2d5da628694e5e45a2b4cc302a846bbc00014c40820233eb026d3ebd4f68c2c9913
cryptapus
pushed a commit
to cryptapus/bitcoin
that referenced
this pull request
May 3, 2021
3589732 Add allowExisting option for name_new. (Daniel Kraft) 8fc3973 Introduce generalised HelpTextBuilder. (Daniel Kraft) Pull request description: As proposed in bitcoin#54, this leverages the new `options` argument to `name_new` to prevent accidental `name_new` transactions where the name exists already. To do this, we add a new `allowExisting` field in the `options` and only allow sending a `name_new` for an existing name if that option is explicitly set by the user. Relative to the current behaviour, this changes the "default" but still keeps the option for power users to override the check and get back the old behaviour. Also, of course, there are no changes on the consensus level. Tree-SHA512: c6e549cfab0ce9578a031b4fd8ec95961e67af2fb0481028ffa1c03444795c647dc004080afc76ccbc6175c30745bad068b755e403c9bfcd2fff2ea218ce9c30
This pull request was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
instead, assign the address to "" account