Page MenuHomePhabricator

Merge #12240: [rpc] Introduced a new `fees` structure that aggregates all sub-field fee types denominated in BCH
ClosedPublic

Authored by nakihito on Sep 25 2019, 18:45.

Details

Summary

7de1de7 Add new fee structure with all sub-fields denominated in BTC (mryandao)

Pull request description:

the denomination for `fee` is current in btc while the other such as `decendentFee` and `ancestorFee` are in satoshis.

Tree-SHA512: e428f6dca1d339f89ab73e38ce5903f5465c46b159069d9bcc3f8b1140fe6657fa49a11abe0088e9f7ba9999f64af72a349a4735bf5eaa61b8e4a185b23543f3

Backport of Core PR12240
https://github.com/bitcoin/bitcoin/pull/12240/

Also included PR13109 and PR13124 (small typo fixes)
https://github.com/bitcoin/bitcoin/pull/13109/
https://github.com/bitcoin/bitcoin/pull/13124/

Test Plan
make check
test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR12240
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7587
Build 13214: Bitcoin ABC Buildbot (legacy)
Build 13213: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Sep 25 2019, 18:45

Applied another typo fix changes.

nakihito retitled this revision from Merge #12240: [rpc] Introduced a new `fees` structure that aggregates all sub-field fee types denominated in BTC to Merge #12240: [rpc] Introduced a new `fees` structure that aggregates all sub-field fee types denominated in BCH.Sep 25 2019, 19:02
jasonbcox requested changes to this revision.Oct 1 2019, 18:44
jasonbcox added inline comments.
doc/release-notes.md
6

Ideally, this would be formatted so that it's readable in 80-char wide terminals. Add some newlines to break this up.

This revision now requires changes to proceed.Oct 1 2019, 18:44

Reformatted release notes.

jasonbcox requested changes to this revision.Oct 1 2019, 21:26
jasonbcox added inline comments.
doc/release-notes.md
10 ↗(On Diff #13308)

This still isn't right. Markdown uses whitespace to group bullet-pointed, multi-line content. See old release notes as examples.

This revision now requires changes to proceed.Oct 1 2019, 21:26

Further release-notes formatting.

This revision is now accepted and ready to land.Oct 1 2019, 22:07