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

api/consensus: support filtering blocks by proposer #764

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

ptrus
Copy link
Member

@ptrus ptrus commented Oct 8, 2024

We have a related request for new api endpoints for signed/proposed blocks for a given validator

Design is here: https://www.figma.com/design/8dwSyYOOm3vlgbvxPGT29p/Block-Explorer-(Post-Launch)?node-id=8694-95348&node-type=frame&t=SLj8iOo49jAnc7nE-0

@ptrus ptrus force-pushed the ptrus/feature/proposed-blocks branch 2 times, most recently from d6b4b43 to 6ec6874 Compare October 8, 2024 12:03
storage/client/client.go Outdated Show resolved Hide resolved
api/spec/v1.yaml Outdated Show resolved Hide resolved
@ptrus ptrus force-pushed the ptrus/feature/proposed-blocks branch from 6ec6874 to 067335a Compare October 9, 2024 10:11
@ptrus ptrus requested a review from Andrew7234 October 9, 2024 10:12
@ptrus ptrus changed the title api/consensus: validator proposed blocks endpoint api/consensus: support filtering blocks by proposer Oct 9, 2024
@ptrus ptrus force-pushed the ptrus/feature/proposed-blocks branch 2 times, most recently from 6cd96aa to 105285a Compare October 9, 2024 10:28
Copy link
Collaborator

@Andrew7234 Andrew7234 left a comment

Choose a reason for hiding this comment

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

Thank you!

storage/client/queries/queries.go Show resolved Hide resolved
@ptrus
Copy link
Member Author

ptrus commented Oct 10, 2024

Hm the E2E test discovered that storing block signers does not currently work correctly when fast-syncing over the consensus blocks, detailed comment here: https://github.com/oasisprotocol/nexus/pull/646/files#r1795395104

@ptrus ptrus force-pushed the ptrus/feature/proposed-blocks branch from 105285a to f1471c2 Compare October 10, 2024 13:11
@ptrus ptrus force-pushed the ptrus/feature/proposed-blocks branch from f1471c2 to bb6da14 Compare November 20, 2024 10:16
@ptrus ptrus merged commit 69cadf5 into main Nov 20, 2024
16 checks passed
@ptrus ptrus deleted the ptrus/feature/proposed-blocks branch November 20, 2024 10:22
@csillag csillag mentioned this pull request Dec 22, 2024
2 tasks
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.

2 participants