Skip to content

fix(nodes): healthy CAN state reads "OK" instead of error-active - #163

Merged
mottihoresh merged 1 commit into
fix/parkedfrom
fix/p162-can-ok
Oct 18, 2026
Merged

mottihoresh merged 1 commit into
fix/parkedfrom
fix/p162-can-ok

Conversation

@mottihoresh

Copy link
Copy Markdown
Contributor

Today: a board panel's CAN state row shows "error-active" for the healthy CAN state (panels.ts:241), which reads like a fault.

After: the healthy state reads "OK"; bus-off and error-passive unchanged. Display text only.

Tests changed on purpose: panels.test.ts (CAN state row) and t18-node-panel-status (rendered row) expect "OK".

Closes #162

@mottihoresh
mottihoresh merged commit 9337aac into fix/parked Oct 18, 2026
3 checks passed
@mottihoresh
mottihoresh deleted the fix/p162-can-ok branch October 18, 2026 02:15
mottihoresh added a commit that referenced this pull request Oct 18, 2026
…s, paused log keeps rows, can state reads ok (#167)

The four parked items kept from the fix track (#135), prioritized
2026-10-18. Item PRs squash-merged into `fix/parked`:
- #163 — a healthy CAN state reads "OK" instead of "error-active".
- #164 — Connection › Connected nodes says "N new board(s) on the bus —
add it/them from the Cockpit." when boards are on the bus but none is in
your cockpit, instead of "No nodes reported".
- #165 — Start with no Gateway (Bridge, or Replay driving the board)
starts and waits for the board with the waiting reason shown, instead of
being silently refused; delivery begins when the Gateway connects.
- #166 — Log › Stream holds rows that arrive while paused and shows them
on Resume (within the row cap), like Events.

Closes #159
Closes #160
Closes #161
Closes #162

BEGIN_COMMIT_OVERRIDE
fix(nodes): healthy CAN state reads "OK" instead of error-active
fix(nodes): connected nodes points to new boards on the bus instead of
saying none are reported
fix(dcs): start with no gateway starts and waits for the board instead
of being refused
fix(log): the stream keeps rows that arrive while paused and shows them
on resume
END_COMMIT_OVERRIDE
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.

1 participant