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

Improvements for custom platform integrations #400

Merged
merged 4 commits into from
Feb 2, 2025

Conversation

matth-x
Copy link
Owner

@matth-x matth-x commented Feb 2, 2025

Simplify some aspects of custom platform integrations:

  • Extend C-API
    • Revise Configurations C-API
    • Add parentIdTag getter in Transactions C-API
    • Move Console Cb and Rng Cb to Platform.h
    • Simplify the usage of custom FilesystemAdapters
    • Add ocpp_addMeterValueInputIntTx to support reading Tx start and stop values
  • Revise SCons support
  • Built-in support for write-only Configurations

Matthias Akstaller and others added 4 commits February 2, 2025 16:58
* Previously, the SConscript added all MO subfolders to the global include paths. However, MO expects only ./src to be on include and the superfluous folders may lead to filename conflicts.

* Remove ArduinoJson from the sources as it's a header-only library and only needs to be on the include path

* Add .c files to the list of source objects
The "AuthorizationKey" config is specified as write-only and thus, may be updated by the server but not reported through the GetConfiguration option.
* Remove Platform setup functions from MicroOcpp_c.h /.cpp which can be found in Platform.h

* Add FilesystemAdapter C-handle

* Expose library lifecycle through `ocpp_is_initialized()`

* Update Configuration C-API

* Update Transaction C-API
* update changelog
* fix CI/CD errors
@matth-x matth-x merged commit 0455ec0 into main Feb 2, 2025
9 of 10 checks passed
@matth-x matth-x deleted the feature/improve-platform-compat branch February 2, 2025 23:28
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.

1 participant