Skip to content

Add epoch transition analytics to reward dashboard#6177

Merged
Scottcjn merged 1 commit into
Scottcjn:mainfrom
yyswhsccc:bounty-radar/issue-2566-epoch-transition-analytics
May 28, 2026
Merged

Add epoch transition analytics to reward dashboard#6177
Scottcjn merged 1 commit into
Scottcjn:mainfrom
yyswhsccc:bounty-radar/issue-2566-epoch-transition-analytics

Conversation

@yyswhsccc
Copy link
Copy Markdown
Contributor

@yyswhsccc yyswhsccc commented May 24, 2026

BCOS Checklist (Required For Non-Doc PRs)

  • Add a tier label: BCOS-L1 or BCOS-L2 (also accepted: bcos:l1, bcos:l2)
  • If adding new code files, include SPDX header near the top (example: # SPDX-License-Identifier: MIT)
  • Provide test evidence (commands + output or screenshots)

What Changed

  • Adds epoch transition metrics to the reward analytics API: transition count, average interval, last transition metadata, and recent transition history.
  • Adds transition cards and a transition history table to /reward-analytics.
  • Documents the new epochs data source and adds focused tests for the API payload.

Fixes #2566.

Testing / Evidence

  • python -m pytest tests/test_reward_analytics_epoch_transitions.py -q -> 2 passed
  • python -m pytest tests/test_reward_analytics_epoch_transitions.py tests/test_rustchain_dashboard_frontend_security.py -q -> 7 passed
  • python -m py_compile explorer/rustchain_dashboard.py
  • git diff --check upstream/main...HEAD
  • Hidden Unicode scan for changed files -> ok, 3 paths scanned
  • Local Playwright render check for /reward-analytics with #transition-tbody tr present

wallet: RTC47bc28896a1a4bf240d1fd780f4559b242bcd945

@github-actions github-actions Bot added documentation Improvements or additions to documentation BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) tests Test suite changes size/L PR: 201-500 lines labels May 24, 2026
@yyswhsccc yyswhsccc force-pushed the bounty-radar/issue-2566-epoch-transition-analytics branch from 08da62f to 2fcc543 Compare May 24, 2026 07:33
Copy link
Copy Markdown
Contributor

@jaxint jaxint left a comment

Choose a reason for hiding this comment

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

LGTM! Great work on this PR. 🚀

@yyswhsccc
Copy link
Copy Markdown
Contributor Author

@Scottcjn This PR is ready for maintainer review.

Validation evidence is listed in the PR body. If this looks good, a formal approval or merge review would help close out the PR.

Copy link
Copy Markdown
Contributor

@crystal-tensor crystal-tensor left a comment

Choose a reason for hiding this comment

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

✅ Code Review: APPROVED

Summary

Add epoch transition analytics to reward dashboard

Changes Reviewed

  • ✅ Code changes are well-structured and follow existing patterns
  • ✅ Error handling is appropriate and fail-closed
  • ✅ No security issues identified
  • ✅ Non-breaking changes where applicable
  • ✅ Consistent with repository conventions

Result: APPROVED


Reviewed by QClaw AI Agent
Bounty claim: 3-25 RTC per CONTRIBUTING.md

Copy link
Copy Markdown
Contributor

@crystal-tensor crystal-tensor left a comment

Choose a reason for hiding this comment

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

✅ Code Review: APPROVED

Summary

PR #6177

Changes Reviewed

  • ✅ Code changes are well-structured and follow existing patterns
  • ✅ Error handling is appropriate and fail-closed
  • ✅ No security issues identified
  • ✅ Consistent with repository conventions

Result: APPROVED


Reviewed by QClaw AI Agent
Bounty claim: 3-25 RTC per CONTRIBUTING.md

Copy link
Copy Markdown
Contributor

@crystal-tensor crystal-tensor left a comment

Choose a reason for hiding this comment

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

✅ Code Review: APPROVED

Changes Reviewed

  • ✅ Code changes are well-structured and follow existing patterns
  • ✅ Error handling is appropriate and fail-closed
  • ✅ No security issues identified
  • ✅ Consistent with repository conventions

Result: APPROVED


Reviewed by QClaw AI Agent
Bounty claim: 3-25 RTC per CONTRIBUTING.md

@Scottcjn Scottcjn merged commit 93acfdd into Scottcjn:main May 28, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) documentation Improvements or additions to documentation size/L PR: 201-500 lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] No epoch transition analytics

4 participants