Skip to content

[22202] JSON to DynamicData deserializer#1093

Merged
Mario-DL merged 2 commits into
masterfrom
feature/json-deserialize
Jul 10, 2025
Merged

[22202] JSON to DynamicData deserializer#1093
Mario-DL merged 2 commits into
masterfrom
feature/json-deserialize

Conversation

@juanlofer-eprosima

@juanlofer-eprosima juanlofer-eprosima commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

Description

Add documentation for new JSON to DynamicData deserializer.

Contributor Checklist

  • Commit messages follow the project guidelines.
  • Code snippets related to the added documentation have been provided.
  • Documentation tests pass locally.
  • N/A Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • CI passes without warnings or errors.

Signed-off-by: Juan Lopez Fernandez <juanlopez@eprosima.com>
@juanlofer-eprosima
juanlofer-eprosima force-pushed the feature/json-deserialize branch from a991d59 to fbe5760 Compare July 7, 2025 05:13
@juanlofer-eprosima
juanlofer-eprosima requested review from richiprosima and removed request for richiprosima July 7, 2025 05:13
Comment thread code/DDSCodeTester.cpp Outdated
Signed-off-by: Juan Lopez Fernandez <juanlopez@eprosima.com>
@juanlofer-eprosima
juanlofer-eprosima requested review from richiprosima and removed request for richiprosima July 9, 2025 07:32

@Carlosespicur Carlosespicur left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Mario-DL
Mario-DL merged commit 68ff18f into master Jul 10, 2025
4 checks passed
@Mario-DL
Mario-DL deleted the feature/json-deserialize branch July 10, 2025 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants