Show Charging page faults from the fault feed

オープン
  • 何か問題がありますか？
#959 コメント 0 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@SimeonChenchev1 がすでに取り組んでいます。

2026年11月6日 から。

評価

この issue はまだ評価されていません。

説明

by: ai-assisted dev work frontend
What

The Charging page's fault display watches a hard-coded list of BMS and charger fault topics and works out when each fault is raised. That list goes stale whenever the car's fault definitions change, which is what #902 had to fix. Make it show BMS and charger faults from the same fault feed the Faults page uses, so it picks up whatever faults scylla-server reports for those CAN nodes.

Diagram
flowchart LR
  subgraph Before
    S1[Siren] --> SC1[scylla-server] -->|"data, per topic"| FD1["Charging fault display<br/>(hard-coded topic list)"]
  end
  subgraph After
    S2[Siren] --> SC2[scylla-server] -->|"faults event"| FS[FaultService]
    FS --> FP[Faults page]
    FS -->|"Bms and Charger nodes"| FD2[Charging fault display]
  end
Requirements
  • The Charging page fault display gets its faults from the same fault feed as the Faults page, not from its own topic list
  • It shows the same BMS and charger faults it shows today
  • A new BMS or charger fault that scylla-server reports appears without any change to the client
  • The fault display's hard-coded topic list and the topic helpers only it uses are removed
  • Desktop and mobile Charging page layouts both show the faults
Notes
  • The Faults page lives in pages/fault-page; FaultService is in services/fault.service.ts. The fault feed is filled from the socket's "faults" event in socket.service.ts.
  • scylla-server decides what counts as a fault in socket_handler.rs. BMS faults are any topic under BMS/Faults/, charger faults any topic under Charger/Box/F_, so both already reach the feed. A fault is marked expired after 8 seconds without being seen again.
  • The feed sends the whole list of recent faults every second, not just new ones. Each item has its node, name, first-seen time, last-seen time and an expired flag.
  • Open question: should expired faults still be listed, maybe greyed out, or hidden? And should the display's reset button still clear the list?
  • Follows the review on PR #954.
主要言語
TypeScript
スター
5
フォーク
1
平均マージ
1日 8時間
マージ済み PR（30日）
35

環境構築

  • Dockerfile・Docker Compose ファイルなし
  • プルリクエストのテンプレートあり
  • コントリビューションガイドなし

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

Northeastern-Electric-Racing/Argos のほかの issue

Northeastern-Electric-Racing/Argos の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。