Skip to content

fix(daemon): added poa block to isBlock check#337

Merged
andreabadesso merged 1 commit intomasterfrom
fix/poa-block-check
Jan 8, 2026
Merged

fix(daemon): added poa block to isBlock check#337
andreabadesso merged 1 commit intomasterfrom
fix/poa-block-check

Conversation

@andreabadesso
Copy link
Copy Markdown
Collaborator

@andreabadesso andreabadesso commented Jan 6, 2026

Motivation

PoA blocks are being stored with height = null in the Ekvlibro mainnet

Acceptance Criteria

  • PoA blocks should be considered in the isBlock method so they get their height set.

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged
  • Make sure either the unit tests and/or the QA tests are capable of testing the new features
  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@andreabadesso andreabadesso self-assigned this Jan 6, 2026
@andreabadesso andreabadesso moved this from Todo to In Progress (Done) in Hathor Network Jan 6, 2026
@andreabadesso andreabadesso added the bug Something isn't working label Jan 6, 2026
@github-project-automation github-project-automation Bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Jan 6, 2026
@andreabadesso andreabadesso merged commit 0a33679 into master Jan 8, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Jan 8, 2026
@andreabadesso andreabadesso deleted the fix/poa-block-check branch January 8, 2026 12:32
@andreabadesso andreabadesso mentioned this pull request Jan 28, 2026
2 tasks
@andreabadesso andreabadesso moved this from Waiting to be deployed to Done in Hathor Network Jan 28, 2026
This was referenced Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants