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

Fix serialization of payloads that contain collection of multiple elements #637

Merged
merged 5 commits into from
May 21, 2024

Conversation

OrangeTux
Copy link
Contributor

@OrangeTux OrangeTux commented May 21, 2024

Serialization of a payload that contains a collection of items fails. Only the last item of the collection appears in the
serialized output. All other collections are not available.

Fixes #635

@OrangeTux OrangeTux requested a review from tropxy May 21, 2024 13:32
Copy link
Collaborator

@OSkrk OSkrk left a comment

Choose a reason for hiding this comment

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

Looking good.

@OrangeTux OrangeTux merged commit 0b3db4e into master May 21, 2024
6 checks passed
@OrangeTux OrangeTux deleted the 635-fix-serialization branch May 21, 2024 19:49
ajmirsky pushed a commit to ajmirsky/ocpp that referenced this pull request Nov 26, 2024
…ments (mobilityhouse#637)

Serialization of a payload that contains a collection of items fails.
Only the last item of the collection appears in the
serialized output. All other collections are not available.

Fixes mobilityhouse#635
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants