Skip to content

Conversation

Alenar
Copy link
Collaborator

@Alenar Alenar commented Oct 30, 2024

Content

This PR modify the aggregator signers/registered/{epoch} route to accept not only numbers for the epoch but also latest. If latest is ask then the route handler will call the aggregator epoch service to obtain the current working epoch of the aggregator.

Plus the explorer "Signers registrations" page is updated to support latest as the epoch in its url.

Note: in order to describe this change in the openapi I had to upgrade to openapi version 3.1 (else I can't describe "const" parameter such as latest).

Pre-submit checklist

  • Branch
    • Tests are provided (if possible)
    • Crates versions are updated (if relevant)
    • CHANGELOG file is updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Issue(s)

Closes #1689

@Alenar Alenar self-assigned this Oct 30, 2024
Copy link

github-actions bot commented Oct 30, 2024

Test Results

    4 files  ±0     51 suites  ±0   10m 4s ⏱️ -25s
1 418 tests +3  1 418 ✅ +3  0 💤 ±0  0 ❌ ±0 
1 629 runs  +3  1 629 ✅ +3  0 💤 ±0  0 ❌ ±0 

Results for commit 82ce6ca. ± Comparison against base commit 814ab15.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@dlachaume dlachaume left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Copy link
Member

@jpraynaud jpraynaud left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Alenar Alenar force-pushed the djo/1689/latest_for_registered_signers branch from c9cd447 to 82ce6ca Compare October 30, 2024 18:28
@Alenar Alenar temporarily deployed to testing-preview October 30, 2024 18:36 — with GitHub Actions Inactive
@Alenar Alenar temporarily deployed to testing-sanchonet October 30, 2024 18:36 — with GitHub Actions Inactive
@Alenar Alenar merged commit dad3dbc into main Oct 30, 2024
47 checks passed
@Alenar Alenar deleted the djo/1689/latest_for_registered_signers branch October 30, 2024 18:39
Alenar added a commit that referenced this pull request Nov 25, 2024
Since #2062 the version of our openapi file is `3.1.1` but the action
that we use (`Legion2/swagger-ui-action@v1`) is configured by default to
use swaggerui version 3 or up and swagger ui is only compatible with
openapi 3.1 since version 5.
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.

Access registered signers for latest epoch in explorer
3 participants