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

backport: Merge bitcoin#(partial) 24355, 24797 #6061

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

vijaydasmp
Copy link

bitcoin backports

Copy link

This pull request has conflicts, please rebase.

@vijaydasmp vijaydasmp force-pushed the bp24_1 branch 3 times, most recently from c455963 to ebbbf6d Compare July 20, 2024 06:32
@vijaydasmp vijaydasmp changed the title backport: Merge bitcoin#24559, 24355, 24797 backport: Merge bitcoin#24355, 24797 Jul 20, 2024
@vijaydasmp vijaydasmp marked this pull request as ready for review July 29, 2024 15:18
@vijaydasmp
Copy link
Author

Hello @UdjinM6 , requesting review

@vijaydasmp vijaydasmp changed the title backport: Merge bitcoin#24355, 24797 backport: Merge bitcoin# (partial) 24355, 24797 Aug 3, 2024
UdjinM6
UdjinM6 previously approved these changes Aug 5, 2024
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK 1a82e12

@vijaydasmp
Copy link
Author

Hello @PastaPastaPasta , please review

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK 8428bba

laanwj and others added 2 commits August 29, 2024 10:14
… macro

1633f5e util, refactor: Add UNIQUE_NAME helper macro (Hennadii Stepanov)

Pull request description:

  This PR replaces repetitive code with a helper macro.

ACKs for top commit:
  laanwj:
    Tested ACK 1633f5e

Tree-SHA512: 5f04e472c5f3184c0a9df75395377c6744bfb2cd8f95f8427c1c5e20daa7d6a9b29e45424b88391fc6326d365907a750ab50fda534b49d1df80dccf0e18467a4
Signed-off-by: Vijay <vijaydas.mp@gmail.com>
…ckchaininfo` RPC

0f7dc89 test: compare `/chaininfo` response with `getblockchaininfo` RPC (brunoerg)

Pull request description:

  The `/chaininfo` REST endpoint gets its infos from `getblockchaininfo` RPC, so this PR adds an `assert_equal` (in `interface_rest`) to ensure both responses are the same. Obs: other endpoints do the same for their respective RPC.

ACKs for top commit:
  0xB10C:
    Concept and Code Review ACK 0f7dc89. Belts-and-spenders.

Tree-SHA512: 51cbcf988090272e406a47dc869710740b74e2222af29c05ddcbf53bd49765cdc59efb525e970867f091b3d2efec4fb13371a342d9e484e51144b760265bc5b8
Signed-off-by: Vijay <vijaydas.mp@gmail.com>
@vijaydasmp
Copy link
Author

Hello @PastaPastaPasta , requesting review

@vijaydasmp vijaydasmp changed the title backport: Merge bitcoin# (partial) 24355, 24797 backport: Merge bitcoin#(partial) 24355, 24797 Sep 2, 2024
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK e5e0b00

@PastaPastaPasta PastaPastaPasta merged commit 0472a07 into dashpay:develop Sep 3, 2024
33 of 34 checks passed
@UdjinM6 UdjinM6 added this to the 21.2 milestone Sep 5, 2024
@UdjinM6 UdjinM6 modified the milestones: 21.2, 22 Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants