acmerobotics / ftc-dashboard

React-based web dashboard designed for FTC
https://acmerobotics.github.io/ftc-dashboard
Other
178 stars 137 forks source link

Fixed issue where dashboard crashes when double marked with config is infinite or NaN #76

Closed Eeshwar-Krishnan closed 2 years ago

Eeshwar-Krishnan commented 2 years ago

someone had this issue thought I would pr it lol

rbrott commented 2 years ago

Nice, this is indeed a common issue. This minimal patch gets the job done but doesn't really fit with my general philosophy of surfacing errors directly instead of silently proceeding. I'd prefer a change like one of the following (in order of increasing effort).