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

Supported Measurands for OCPP1.6 #492

Merged
merged 2 commits into from
Mar 5, 2024
Merged

Conversation

Pietfried
Copy link
Contributor

@Pietfried Pietfried commented Feb 25, 2024

Describe your changes

  • Added internal configuration key: SupportedMeasurands
  • Initializing supported measurands on startup (was an open TODO)
  • Measurands that are configured that are not part of the configured supported measurands are rejected or application crashes on startup if misconfigured

Issue ticket number and link

#485

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

* Initializing supported measurands on startup
* Measurands that are configured that are not part of the configured supported measurands are rejected or application crashes on startup if misconfigured

Signed-off-by: pietfried <[email protected]>
@Pietfried
Copy link
Contributor Author

Adding a callback to allow libocpp to request the supported measurands from the user application can be done as an additional step.

@Pietfried Pietfried linked an issue Feb 26, 2024 that may be closed by this pull request
Signed-off-by: pietfried <[email protected]>
@Pietfried Pietfried merged commit 272be7a into main Mar 5, 2024
4 checks passed
@Pietfried Pietfried deleted the feature/v16-supported-measurands branch March 5, 2024 21:48
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.

ChangeConfiguration.req is accepted for unsupported measurands
3 participants