Skip to content

Conversation

@Dalachowsky
Copy link

Describe your changes

Added messages related to cable check.
This PR also introduces EV/EVSE status structs and EVSEProcessing enum.

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

Kacper Dalach added 2 commits December 17, 2025 09:14
These structs are a common part in many messages.

Signed-off-by: Kacper Dalach <kdalach@elektrometal.com.pl>
Signed-off-by: Kacper Dalach <kdalach@elektrometal.com.pl>
@Dalachowsky Dalachowsky changed the title Feat/adding iso2 support cable check libiso15118: Add -2 cable check messages Dec 17, 2025
Signed-off-by: Kacper Dalach <kdalach@elektrometal.com.pl>
@Dalachowsky Dalachowsky force-pushed the feat/adding-iso2-support-cable-check branch from 20f4fbd to 9272103 Compare December 17, 2025 08:31
@Dalachowsky
Copy link
Author

I have also noticed that for -20 cable check response gets tested for each value of Processing enum. Since enum conversion is not handled by libiso15118 I don't see a reason to introduce these tests here.
What I would introduce would be tests that check if optional fields are properly handled, since this logic is introduced in libiso15118.

@Dalachowsky
Copy link
Author

This PR was superseeded by #1679

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.

1 participant