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 2.0 validate ChargingStationMaxProfile #10

Conversation

folkengine
Copy link

Describe your changes

Handles K01.FR.38 and K04.FR.03 from the OCPP 2.0.1 specification.

K01.FR.38: When chargingProfilePurpose = ChargingStationMaxProfilechargingProfileKind SHALL NOT be Relative

K04.FR.03: The optional ChargingSchedule field minChargingRate MAY be used by the Charging Station to optimize the power distribution between the EVSEs.

Issue ticket number and link

K01 - Charging Station Max Profile Validation #99

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

@folkengine folkengine marked this pull request as ready for review April 26, 2024 17:29
Copy link

@couryrr-afs couryrr-afs left a comment

Choose a reason for hiding this comment

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

LGTM - ☕

@christopher-davis-afs
Copy link

With the new workflow, I wonder if this should be squashed to one commit, since the other two commits are modifications to the changes introduced in the first.

Other than that, LGTM

@folkengine folkengine merged commit bc38476 into feature/k01-validate-profile Apr 26, 2024
2 checks passed
@folkengine folkengine deleted the folkengine/k01_validate_ChargingStationMaxProfile_III branch April 26, 2024 18:04
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.

3 participants