Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding new iso15118_extension interface #1022

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

SebaLukas
Copy link
Contributor

Describe your changes

With this PR there is a new interface between ISO and OCPP modules. Now the ISO and OCPP modules can communicate directly and no longer have to communicate via the EvseManager. This PR does not add any new cmds and vars.
This is done in a separate PR.

Issue ticket number and link

Until now, the CertificateInstallation messages from the ISO module were forwarded to the OCPP module via the EvseManager. The response from the CSMS also had to be published via the EvseManager to the ISO modules. With OCPP2.1, there is significantly more information that needs to be forwarded from the ISO module to OCPP. Therefore, the path via the EvseManager is cut and a new interface between OCPP and the ISO module is introduced.

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

Pietfried and others added 7 commits January 23, 2025 10:28
…ules have been modified accordingly

Signed-off-by: Sebastian Lukas <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
Signed-off-by: Piet Gömpel <[email protected]>
@hikinggrass hikinggrass added the post-release Tag that this PR should not go into the current merge window for the upcoming release label Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
post-release Tag that this PR should not go into the current merge window for the upcoming release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants