Skip to content

fix: odometer missing in chargelog - #3683

Draft
rleidner wants to merge 1 commit into
openWB:masterfrom
rleidner:fix_chargelog_odometer
Draft

fix: odometer missing in chargelog#3683
rleidner wants to merge 1 commit into
openWB:masterfrom
rleidner:fix_chargelog_odometer

Conversation

@rleidner

Copy link
Copy Markdown
Collaborator

fix for issue #3677

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR targets issue #3677 by attempting to propagate the EV’s odometer value into the chargepoint’s published “connected vehicle” state so it can be consumed elsewhere (e.g., UI/chargelog).

Changes:

  • Copy vehicle.data.get.odometer into chargepoint.data.get.connected_vehicle.soc.odometer when publishing connected-vehicle SOC data.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

self.data.get.connected_vehicle.soc.fault_state = vehicle.data.get.fault_state
self.data.get.connected_vehicle.soc.fault_str = vehicle.data.get.fault_str
self.data.get.connected_vehicle.soc.range = vehicle.data.get.range
self.data.get.connected_vehicle.soc.odometer = vehicle.data.get.odometer
@rleidner
rleidner marked this pull request as draft July 21, 2026 10:56
@rleidner

Copy link
Copy Markdown
Collaborator Author

The user who reported this issue initially later added that it worked again without change.
copilit is right, it should work without that change.
I set this PR to draft for now and will do more testing/analysis.

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.

2 participants