Skip to content

Conversation

@td202
Copy link
Contributor

@td202 td202 commented Sep 19, 2024

Purpose

Closes #1240

This fixes the timing information for the next payday returned by the GetTokenomicsInfo query. Instead of basing it on the genesis block time, the time is computed based on the next epoch transition time. This is more accurate, since after the P6->P7 update, the first epoch (epoch 0) is already considered complete.

Changes

  • Revise the computation of the next payday time, as described above.
  • Bump node version for new release.

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

Copy link
Contributor

@drsk0 drsk0 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@td202 td202 merged commit 2e6857b into main Oct 22, 2024
2 of 3 checks passed
@td202 td202 deleted the fix-next-payday branch October 22, 2024 10:56
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.

Reported next payday time is off by one epoch

5 participants