Skip to content

Conversation

@mlitre
Copy link
Contributor

@mlitre mlitre commented Jul 23, 2025

Describe your changes

V2X variables were set globally when they were supposed to be evse specific.

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
  • If OCPP 2.0.1 or OCPP2.1: I have updated the OCPP 2.x status document
  • I read the contribution documentation and made sure that my changes meet its requirements

@mlitre mlitre added bug Something isn't working OCPP2.1 labels Jul 23, 2025
@mlitre mlitre force-pushed the fix/make-v2x-component-evse-specific branch from 97cba5d to de214a2 Compare July 24, 2025 07:54
Copy link
Contributor

@Pietfried Pietfried left a comment

Choose a reason for hiding this comment

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

lgtm, see two comments in line, leaving it up to you to apply

@mlitre mlitre force-pushed the fix/make-v2x-component-evse-specific branch 2 times, most recently from f8b0593 to 91b4830 Compare August 4, 2025 08:27
@mlitre mlitre force-pushed the fix/make-v2x-component-evse-specific branch from 91b4830 to cae4a71 Compare August 4, 2025 14:37
@mlitre mlitre requested a review from Pietfried August 4, 2025 14:39
@mlitre mlitre merged commit 54d27c5 into main Aug 6, 2025
10 checks passed
@mlitre mlitre deleted the fix/make-v2x-component-evse-specific branch August 6, 2025 14:51
stephenredhat pushed a commit to stephenredhat/libocpp that referenced this pull request Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working OCPP2.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants