TODO for release notes 0.14.0

laanwj commented at 1:08 pm on August 4, 2016: member

Post things here that need explicit mention in 0.14 release notes

I’d strongly prefer for pulls to just update doc/release-notes.md directly, but if they somehow slip by, mention them here.

Overview:

ZMQ on Windows (#8238) (done in wiki)
Changed RPC calls:
- getaddednodeinfo got rid of first argument (#8272) (done in wiki)
- getinfo was deprecated (but not yet removed) (#8780) (done in wiki)
- preciousblock RPC call #6996
- importmulti RPC: #7551
- getmemoryinfo #8753
- `setnetworkactive RPC (#8996)
Mention that the GUI confirmation target has been changed from the default of 25 to 6 (#9036).
Mempool dump to disk (#8448) (done in wiki)
Several memory usage improvements
- (shared pointers, LockedPool #8753)
- Share unused mempool space with coin cache (#8610)
Qt RPC nested commands: #7783 (done in wiki)
Qt sensitive data filter: #8877 (done in wiki)
Removal of priority estimation (#7730).
Address relay changes (#8594, #8661, #8740).
Named RPC arguments (#8811).
Manual block file pruning (#7871)
Deprecate use of getrawtransaction for confirmed transactions with unspent outputs without -txindex (#9520).

Note: we’re doing an experiment with editing the release notes draft in a github wiki. The most up to date version of the release notes is thus at https://github.com/bitcoin-core/bitcoin-devwiki/wiki/0.14.0-Release-notes . Feel free to edit them there if you want to add any of these items.

laanwj added this to the milestone 0.14 on Aug 4, 2016

laanwj added the label Docs and Output on Aug 4, 2016

fanquake commented at 8:06 am on August 6, 2016: member

ZMQ on Windows from #8238

MarcoFalke commented at 8:41 am on September 14, 2016: member

* #8699: Remove createwitnessaddress RPC command

MarcoFalke commented at 8:42 am on September 14, 2016: member

Is the release schedule for 0.14 out yet? (I couldn’t find it anywhere)

laanwj commented at 8:56 am on September 14, 2016: member

@MarcoFalke thanks for the reminder, see #8719

laanwj commented at 1:04 pm on September 26, 2016: member

[rpc] Deprecate getinfo #8780

laanwj commented at 1:38 pm on September 29, 2016: member

As for deprecating getinfo, the following table may be useful for in the release notes

`getinfo` field	Moved to
`"version"`	`getnetworkinfo()["version"]`
`"protocolversion"`	`getnetworkinfo()["protocolversion"]`
`"walletversion"`	`getwalletinfo()["walletversion"]`
`"balance"`	`getwalletinfo()["balance"]`
`"blocks"`	`getblockchaininfo()["blocks"]`
`"timeoffset"`	`getnetworkinfo()["timeoffset"]`
`"connections"`	`getnetworkinfo()["connections"]`
`"proxy"`	`getnetworkinfo()["networks"][0]["proxy"]`
`"difficulty"`	`getblockchaininfo()["difficulty"]`
`"testnet"`	`getblockchaininfo()["chain"] == "test"`
`"keypoololdest"`	`getwalletinfo()["keypoololdest"]`
`"keypoolsize"`	`getwalletinfo()["keypoolsize"]`
`"unlocked_until"`	`getwalletinfo()["unlocked_until"]`
`"paytxfee"`	`getwalletinfo()["paytxfee"]`
`"relayfee"`	`getnetworkinfo()["relayfee"]`
`"errors"`	`getnetworkinfo()["warnings"]`

laanwj commented at 1:40 pm on September 29, 2016: member

#8699: Remove createwitnessaddress RPC command

This is one for the 0.13.1 release notes, as it was backported in #8815.

jonasschnelli commented at 7:44 pm on October 18, 2016: contributor

preciousblock RPC call #6996

laanwj commented at 7:06 am on October 20, 2016: member

importmulti RPC: #7551

jonasschnelli commented at 12:32 pm on October 28, 2016: contributor

Mention that the GUI confirmation target has been changed from the default of 25 to 2 #8989 (or maybe 6 after #9036)

laanwj commented at 11:37 am on November 2, 2016: member

Yes, also the RPC default will go to 6 if #9036.

New RPC: getmemoryinfo #8753

MarcoFalke added the label Needs release notes on Dec 2, 2016

MarcoFalke commented at 3:56 pm on December 2, 2016: member

Created a label “Needs release notes” for pulls to be tagged with. Could be useful?

laanwj commented at 11:19 am on December 3, 2016: member

@MarcoFalke yes good idea

sipa commented at 0:13 am on January 10, 2017: member

GUI now has a modal out of sync layer (#8371).

sipa commented at 0:17 am on January 10, 2017: member

Share unused mempool space with coin cache (#8610).

sipa commented at 0:18 am on January 10, 2017: member

Mempool dump to disk (#8448).

sipa commented at 0:19 am on January 10, 2017: member

Change default confirm target from 2 to 6 (#9036).

sipa commented at 0:21 am on January 10, 2017: member

Several memory usage improvements (shared pointers, LockedPool #8753)

sipa commented at 0:29 am on January 10, 2017: member

togglenetwork RPC (#8996).

sipa commented at 0:32 am on January 10, 2017: member

Update to LevelDB 1.19 (#8613).

sdaftuar commented at 3:24 pm on January 10, 2017: member

estimatefee for a target of 1 block has been disabled (this part was merged into 0.13.2 as well), and the gui slider will stop at 2 now (new for 0.14) (#9239)

jonasschnelli commented at 3:48 pm on January 10, 2017: contributor

Qt RPC nested commands: https://github.com/bitcoin/bitcoin/pull/7783

jonasschnelli commented at 3:49 pm on January 10, 2017: contributor

Qt sensitive data filter: https://github.com/bitcoin/bitcoin/pull/8877

sipa commented at 4:55 pm on January 10, 2017: member

Return txid even if ATMP fails for new transaction (#9302). Prefer coins that have fewer ancestors (#9262).

sipa commented at 5:00 pm on January 10, 2017: member

Removal of priority estimation (#7730).

sipa commented at 5:06 pm on January 10, 2017: member

Address relay changes (#8594, #8661, #8740).

sipa commented at 5:08 pm on January 10, 2017: member

Named RPC arguments (#8811).

luke-jr commented at 5:10 pm on January 10, 2017: member

#9263 needs to be merged in

laanwj commented at 5:21 pm on January 10, 2017: member

Update to LevelDB 1.19 (#8613).

Do you have a specific reason this needs special mention in the release notes? (besides a normal changelog line) Like other dependency upgrades, it’s not a user-facing change.

MarcoFalke commented at 5:22 pm on January 10, 2017: member

Return txid even if ATMP fails for new transaction (#9302). Prefer coins that have fewer ancestors (#9262).

Those are already in 0.13 so don’t need to be put in the notes for 0.14?

laanwj commented at 5:24 pm on January 10, 2017: member

Those are already in 0.13 so don’t need to be put in the notes for 0.14?

Indeed, good catch. Release notes are a diff to the last minor release on previous major. I guess an exception could be made if it was somehow forgotten there.

sipa commented at 5:47 pm on January 10, 2017: member

Update to LevelDB 1.19 (#8613).

Do you have a specific reason this needs special mention in the release notes? (besides a normal changelog line) Like other dependency upgrades, it’s not a user-facing change.

Fair enough.

laanwj commented at 1:30 pm on January 11, 2017: member

#7871: Manual block file pruning

fanquake commented at 1:54 am on January 12, 2017: member

I’ve updated to top post to include everything added here so far.

sipa commented at 12:24 pm on January 12, 2017: member

#9520 Deprecate use of getrawtransaction for confirmed transactions with unspent outputs without -txindex.

laanwj commented at 8:53 am on January 17, 2017: member

Note: we’re doing an experiment with editing the release notes draft in a github wiki. The most up to date version of the release notes is thus at https://github.com/bitcoin-core/bitcoin-devwiki/wiki/0.14.0-Release-notes . Feel free to edit them there if you want to add any of these items.

MarcoFalke commented at 8:43 pm on January 19, 2017: member

#8456 bumpfee

kallewoof commented at 12:48 pm on January 30, 2017: member

Maybe mentioning changes to listsinceblock as these now use fork point rather than the relative height of the reorged block vs current main chain (#9516).

I’m also confused by the wording “This is a new major version release”, since traditionally, the version is MAJOR.MINOR.PATCH, and this is thus a new MINOR version release. Perhaps “major” is referring to a major change in codebase only?

laanwj commented at 12:59 pm on January 30, 2017: member

since traditionally, the version is MAJOR.MINOR.PATCH, and this is thus a new MINOR version release

Traditionally for other projects perhaps. In bitcoin core we’ve always used “major” for 0.x.0 and “minor” for 0.x.y. To be honest I don’t think that’s going to ever change, unless for some freak reason the first number ever gets bumped, but that’s a completely different discussion that doesn’t belong here.

MarcoFalke commented at 1:01 pm on January 30, 2017: member

We might just drop the 0. prefix if it is always present and thus, count releases like firefox and similar. But agree it is off topic here.

MarcoFalke commented at 1:03 pm on January 30, 2017: member

Could make sense to sync the release notes from the wiki to our repo in a pull request, so we can see if all looks fine.

morcos commented at 7:58 pm on February 9, 2017: member

~~Need to add explanation of at least some of the new minimum fees (incrementalRelayFee, dustRelayFee and blockMinTxFee) as well as the fact that it is now unnecessary to set minRelayTxFee.~~ (done)

MarcoFalke commented at 10:48 am on February 25, 2017: member

bumpfee is still missing?

laanwj commented at 12:48 pm on February 28, 2017: member

#9878 added mention of bumpfee (thanks @ryanofsky), I think this is done, closing.

laanwj closed this on Feb 28, 2017

sipa removed the label Needs release notes on Aug 14, 2018

DrahtBot locked this on Sep 8, 2021

TODO for release notes 0.14.0 #8455