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

feat(rpc): Tweak protx info to be able to show info for a specific block #4738

Merged
merged 5 commits into from
Feb 7, 2024

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Mar 23, 2022

No description provided.

@UdjinM6 UdjinM6 added this to the 18 milestone Mar 23, 2022
@github-actions
Copy link

This pull request has conflicts, please rebase.

@thephez
Copy link
Collaborator

thephez commented Aug 1, 2022

Is this going to be be part of a future release now or is it not required anymore?

@PastaPastaPasta
Copy link
Member

PastaPastaPasta commented Aug 1, 2022

@UdjinM6 can you get this up to date and add tests? (should also add release notes)

@UdjinM6 UdjinM6 removed this from the 18 milestone Oct 17, 2022
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

This pull request has conflicts, please rebase.

@UdjinM6 UdjinM6 changed the title rpc: Tweak protx info to be able to show info for a specific block feat(rpc): Tweak protx info to be able to show info for a specific block Oct 9, 2023
@UdjinM6 UdjinM6 added the RPC Some notable changes to RPC params/behaviour/descriptions label Oct 9, 2023
@UdjinM6 UdjinM6 marked this pull request as ready for review October 9, 2023 16:22
@UdjinM6
Copy link
Author

UdjinM6 commented Oct 10, 2023

1 year later

@PastaPastaPasta With #5607 merged and a fixed BuildDMNListEntry implementation this should be (finally) working correctly in all cases I think. Also added tests and release notes.

@UdjinM6
Copy link
Author

UdjinM6 commented Dec 16, 2023

@knst Applied most of suggestions, thanks! 👍

knst
knst previously approved these changes Dec 17, 2023
Copy link
Collaborator

@knst knst left a comment

Choose a reason for hiding this comment

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

utACK
\

ogabrielides
ogabrielides previously approved these changes Feb 7, 2024
Copy link
Collaborator

@ogabrielides ogabrielides left a comment

Choose a reason for hiding this comment

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

utACK

@UdjinM6
Copy link
Author

UdjinM6 commented Feb 7, 2024

7a00bfe#r138383175

fixed in 9f72012

Copy link
Collaborator

@knst knst left a comment

Choose a reason for hiding this comment

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

utACK

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 for squash merge

@PastaPastaPasta PastaPastaPasta merged commit 3a2bcb1 into dashpay:develop Feb 7, 2024
7 of 10 checks passed
@thephez
Copy link
Collaborator

thephez commented Feb 7, 2024

Should this be under the 20.1 milestone?

@UdjinM6 UdjinM6 added this to the 20.1 milestone Feb 7, 2024
@UdjinM6
Copy link
Author

UdjinM6 commented Feb 7, 2024

Good catch! fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RPC Some notable changes to RPC params/behaviour/descriptions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants