pablobuenaposada / HonDash

Dashboard for Hondata K-Pro & S300 ECUs
GNU Affero General Public License v3.0
66 stars 28 forks source link

React arj #218

Closed AdamRJonesHPD closed 3 years ago

pablobuenaposada commented 3 years ago

what happened @AdamRJonesHPD ?

AdamRJonesHPD commented 3 years ago

what happens @AdamRJonesHPD ?

I made a mess out of things, then didn't look at it for a couple of months, then decided to blow it out and try to re-learn everything! I couldn't even understand my own notes anymore.

I had all available values streaming over the websocket but somehow ended up back at the base packet. Trying to sort that out now and I might get back into trying to build a react app

pablobuenaposada commented 3 years ago

that would be awesome, I see you added analog ports too, I will double-check them and try to add them to the project in the next days

AdamRJonesHPD commented 3 years ago

My injduty formula is jacked but I will sort that out later.

What is your process for adding fields? This is how I approached...

Backend/devices:

Backend/setup_validator:

Backend:

am I missing a step?

pablobuenaposada commented 3 years ago

that seems correct to me! I would suggest to make only a pull request adding one new value and the we figure out all is needed with a smaller scope