Skip to content

merge bitcoin#14700...#16854: backports#4606

Merged
PastaPastaPasta merged 6 commits into
dashpay:developfrom
kwvg:portsportsports
Dec 21, 2021
Merged

merge bitcoin#14700...#16854: backports#4606
PastaPastaPasta merged 6 commits into
dashpay:developfrom
kwvg:portsportsports

Conversation

@kwvg

@kwvg kwvg commented Dec 4, 2021

Copy link
Copy Markdown
Collaborator

No description provided.

@PastaPastaPasta PastaPastaPasta added this to the 18 milestone Dec 5, 2021
Comment thread test/functional/rpc_psbt.py Outdated
Comment thread test/functional/feature_block.py Outdated
@thephez thephez added the RPC Some notable changes to RPC params/behaviour/descriptions label Dec 7, 2021

@PastaPastaPasta PastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, one comment

utACK for merge via merge commit

Comment thread src/rpc/rawtransaction.cpp Outdated
@kwvg
kwvg requested a review from PastaPastaPasta December 12, 2021 08:44
@UdjinM6 UdjinM6 changed the title merge bitcoin#14719...#16854: backports merge bitcoin#14700...#16854: backports Dec 12, 2021
UdjinM6
UdjinM6 previously approved these changes Dec 12, 2021

@UdjinM6 UdjinM6 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

Comment thread src/rpc/rawtransaction.cpp Outdated
Comment thread test/functional/rpc_psbt.py Outdated
@kwvg
kwvg force-pushed the portsportsports branch 2 times, most recently from 11c5fc5 to d81793f Compare December 13, 2021 05:40
Comment thread src/rpc/rawtransaction.cpp Outdated
Comment on lines 1449 to 1445

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

13932: hmmm... doesn't this mean that in our case utxoupdatepsbt simply does nothing all the time?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it exists only to exist but I presume that its absence may be an unexpected API deviation so it was added in despite practically being a NOP.

@github-actions

Copy link
Copy Markdown

This pull request has conflicts, please rebase.

Comment thread test/functional/rpc_psbt.py Outdated
@kwvg
kwvg requested a review from UdjinM6 December 21, 2021 07:17

@UdjinM6 UdjinM6 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@PastaPastaPasta PastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK for merging via merge commit

@PastaPastaPasta
PastaPastaPasta merged commit e95c13c into dashpay:develop Dec 21, 2021
UdjinM6 added a commit to UdjinM6/dash that referenced this pull request Dec 30, 2021
@kwvg
kwvg deleted the portsportsports branch July 18, 2023 11:40
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Nov 16, 2023
gades pushed a commit to piratecash/pirate that referenced this pull request Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bitcoin Backport RPC Some notable changes to RPC params/behaviour/descriptions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants