fleet-telemetry icon indicating copy to clipboard operation
fleet-telemetry copied to clipboard

Add Detailed Charge State field

Open agbpatro opened this issue 1 year ago • 1 comments

Description

Adding a detailed charge state in the payload If the value is present in string, convert into respective enum

Fixes # (issue) https://github.com/teslamotors/fleet-telemetry/issues/123

Type of change

Please select all options that apply to this change:

  • [X] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] Documentation update

Checklist:

Confirm you have completed the following steps:

  • [X] My code follows the style of this project.
  • [X] I have performed a self-review of my code.
  • [X] I have made corresponding updates to the documentation.
  • [X] I have added/updated unit tests to cover my changes.
  • [ ] I have added/updated integration tests to cover my changes.

agbpatro avatar Apr 16 '24 21:04 agbpatro

FYI I tried to use this new field, however when I attempt to reference this field in a vehicle configuration (POST /api/1/vehicles/fleet_telemetry_config) I receive the following error:

{
  response: null,
  error: 'Unknown field DetailedChargeState',
  error_description: '',
  txid: 'd4cd7d67edf5e099f0f4e46d7e7af102'
}

jbanyer avatar May 09 '24 23:05 jbanyer

@agbpatro Any ETA on this? Super important field.

stx avatar Aug 12 '24 18:08 stx

Closing this as it's addressed in #206. Vehicles will be capable of sending this field in a future firmware release.

patrickdemers6 avatar Sep 16 '24 17:09 patrickdemers6