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

Feature/706 ocpp 201 california pricing requirements #757

Merged
merged 26 commits into from
Sep 24, 2024

Conversation

maaikez
Copy link
Contributor

@maaikez maaikez commented Aug 22, 2024

Describe your changes

OCPP 201 california pricing requirements. This includes display message and tariffandcost

Issue ticket number and link

#706

Checklist before requesting a review

…pricing for 2.0.1

Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
…o feature/706-ocpp-201-california-pricing-requirements
…s and add more logging in the callback validate function. Implement cost and tariff from transactioneventresponse.

Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
…dd multilanguage for california pricing.

Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
…ve conversions to everest core. Change transaction id to identifier type. Start with metervalue triggers.

Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
… Some changes in variables with a wrong name.

Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
@Pietfried Pietfried linked an issue Sep 10, 2024 that may be closed by this pull request
…x some bugs, mainly with triggers.

Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
…e in utils). Some comments and also send power active measurand to be sure (if it is there anyway)

Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
@maaikez maaikez marked this pull request as ready for review September 12, 2024 16:28
@maaikez
Copy link
Contributor Author

maaikez commented Sep 12, 2024

Made this a pull request now. I have to merge master still. And I have a question:

Should the metervalue triggers be stored to the database as well? So if libocpp crashes, the triggers will be stored?
It feels a bit too much but on the other hand, it is possible that it crashes and then no triggers will be sent anymore. Only the regular metervalues.

Almost all is (partly) tested with ocpp-testing, except the authorizeresponse message.

Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
Signed-off-by: Maaike Zijderveld, iolar <[email protected]>
@maaikez maaikez force-pushed the feature/706-ocpp-201-california-pricing-requirements branch from 8e7a9fb to a47c677 Compare September 23, 2024 13:24
include/ocpp/v201/charge_point_callbacks.hpp Outdated Show resolved Hide resolved
lib/ocpp/v201/charge_point.cpp Outdated Show resolved Hide resolved
@maaikez maaikez merged commit af2fcc4 into main Sep 24, 2024
3 of 4 checks passed
@maaikez maaikez deleted the feature/706-ocpp-201-california-pricing-requirements branch September 24, 2024 11:23
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.

OCPP 2.0.1 California Pricing Requirements: implement callbacks
2 participants