Skip to content

fix: wrong output type for GetCardanoDatabaseListQuery#2843

Merged
Alenar merged 2 commits into
mainfrom
djo/2827/fix/wrong-output-type-for-get_cdb_list_query
Dec 5, 2025
Merged

fix: wrong output type for GetCardanoDatabaseListQuery#2843
Alenar merged 2 commits into
mainfrom
djo/2827/fix/wrong-output-type-for-get_cdb_list_query

Conversation

@Alenar

@Alenar Alenar commented Dec 5, 2025

Copy link
Copy Markdown
Collaborator

Content

This PR fix the GetCardanoDatabaseListQuery in mithril-aggregator-client, it was not returning the expected type CardanoDatabaseSnapshotListMessage but CardanoDatabaseDigestListMessage instead.

Pre-submit checklist

  • Branch
    • Tests are provided (if possible)
    • Crates versions are updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • All check jobs of the CI have succeeded
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested
  • Documentation
    • No new TODOs introduced

Issue(s)

Relates to #2827

@Alenar
Alenar requested a review from jpraynaud December 5, 2025 09:36
@Alenar Alenar self-assigned this Dec 5, 2025
@github-actions

github-actions Bot commented Dec 5, 2025

Copy link
Copy Markdown

Test Results

    4 files  ±0    168 suites  ±0   23m 27s ⏱️ +14s
2 276 tests ±0  2 276 ✅ ±0  0 💤 ±0  0 ❌ ±0 
7 145 runs  ±0  7 145 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a724d6e. ± Comparison against base commit 8cd976a.

@Alenar
Alenar temporarily deployed to testing-preview December 5, 2025 09:47 — with GitHub Actions Inactive

@jpraynaud jpraynaud 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

@Alenar
Alenar merged commit 977dfe5 into main Dec 5, 2025
48 checks passed
@Alenar
Alenar deleted the djo/2827/fix/wrong-output-type-for-get_cdb_list_query branch December 5, 2025 09:52
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