Skip to content

feat(dashboard): added bandwidth graph - #5218

Open
Je3r wants to merge 1 commit into
freenet:mainfrom
Je3r:bandwidth-graph
Open

feat(dashboard): added bandwidth graph#5218
Je3r wants to merge 1 commit into
freenet:mainfrom
Je3r:bandwidth-graph

Conversation

@Je3r

@Je3r Je3r commented Aug 7, 2026

Copy link
Copy Markdown

Fix #5152

Added a graph showing the last minute of upload and download bandwidth in the dashboard.

The code was written by GPT 5.6 Sol (Codex)

@CLAassistant

CLAassistant commented Aug 7, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Rule Review: No issues found

Rules checked: git-workflow.md, code-style.md, testing.md
Files reviewed: 6

The Rust-side change is a small, additive format! extension in cards.rs (numeric fields only, no injection risk) plus one new regression-style test in home_page.rs — neither triggers any of the applicable code-style.md patterns (no .unwrap() in production code, no spawns, no retry loops, no biased;, no time/rng/socket usage, no persisted config fields). The bulk of the change is browser-side JavaScript (dashboard.js, bandwidth_chart.test.mjs) and CSS, which fall outside the Rust-scoped code-style.md (**/*.rs) and outside the Rust-specific concerns of testing.md (TimeSource/GlobalRng/SimulationSocket). The new .mjs test suite is unusually thorough for the areas testing.md's "edge cases and boundary conditions" section calls out — zero/negative/invalid counters, BigInt overflow, backward clocks, counter resets, window gaps, empty/partial history, and viewport-edge tooltip clamping are all covered. The commit message follows conventional-commits format. No PR-description content was available to review that section of git-workflow.md.

No rule violations detected.


Rule review against .claude/rules/. WARNING findings block merge.

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.

Bandwidth usage graph in local dashboard

2 participants