Skip to content
This repository was archived by the owner on Apr 27, 2026. It is now read-only.

Reserved -> Occupied transition in OCPP2.0.1#1025

Merged
Pietfried merged 1 commit intomainfrom
bugfix/reservation-state-transitions
Mar 31, 2025
Merged

Reserved -> Occupied transition in OCPP2.0.1#1025
Pietfried merged 1 commit intomainfrom
bugfix/reservation-state-transitions

Conversation

@Pietfried
Copy link
Copy Markdown
Contributor

@Pietfried Pietfried commented Mar 28, 2025

Describe your changes

Calling reservation cleared when a transaction starts. Allows the state machine to move from Reserved to Occupied when transactions start. Changes are required because of this change in everest-core:
EVerest/EVerest#1108

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

…te machine to move from Reserved to Occupied when transactions start

Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried Pietfried marked this pull request as ready for review March 28, 2025 14:22
@Pietfried Pietfried merged commit ae8f7fb into main Mar 31, 2025
10 checks passed
@Pietfried Pietfried deleted the bugfix/reservation-state-transitions branch March 31, 2025 06:57
Pietfried added a commit that referenced this pull request Nov 19, 2025
* revert ChangeAvailability scheduled for reserved state (#1024)

Signed-off-by: Martin Litre <[email protected]>

* Calling reservation cleared when a transaction starts. Allows the state machine to move from Reserved to Occupied when transactions start (#1025)

Signed-off-by: Piet Gömpel <[email protected]>

---------

Signed-off-by: Martin Litre <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
Co-authored-by: Martin Litre <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants