Skip to content

fix(analytics): return total trade count - #11

Merged
tingtonglls merged 2 commits into
mainfrom
fix/public-analytics-count
Aug 14, 2026
Merged

tingtonglls merged 2 commits into
mainfrom
fix/public-analytics-count

Conversation

@tingtonglls

Copy link
Copy Markdown
Contributor

修正首页公开指标中的交易笔数统计。

主要改动:

  • 将 netSettlementCount 更名为 totalTradeCount
  • 使用当前结算窗口的原始交易数量 windowTrades.length
  • 保留原始交易总金额、节省流动性和义务减少比例的计算
  • 更新相关测试名称和assert

四项公开指标现在分别表示:

  • 所有原始交易金额总和
  • 所有原始交易笔数
  • 原始交易总金额减去实际净结算金额
  • 节省流动性占原始交易总金额的整数百分比

验证:

  • forge fmt --check 通过
  • forge test --match-path test/unit/Netting.t.sol -vvv 通过
  • forge test -vvv 通过

@tingtonglls
tingtonglls requested a review from Nero978 August 14, 2026 12:22
@tingtonglls
tingtonglls merged commit be515fd into main Aug 14, 2026
1 check passed
@tingtonglls
tingtonglls deleted the fix/public-analytics-count branch August 14, 2026 12:37
@github-actions

Copy link
Copy Markdown

🎉🎉🎉 Congratulations on your new release! 🎉🎉🎉

We have created a release draft of the new version, please go to Release to publish it! ✨

@Nero978 Nero978 added the bug Something isn't working label Aug 20, 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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants