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

Allow Complex JSON as Template input #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Sefriol
Copy link

@Sefriol Sefriol commented Jun 14, 2023

Purpose

  • Previously nested and array JSON was converted to flat dictionary if it were not provided as a string to telemetry. This allows to store different and complex payloads much more nicely.

Does this introduce a breaking change?

[ ] Yes
[ x ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ x ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone git@github.com:Sefriol/Iot-Telemetry-Simulator.git
cd Iot-Telemetry-Simulator
git checkout feature/json-template
  • Test the code

Run the unit tests.

What to Check

Verify that all tests pass

Other Information

I had only Net6.0 and 7.0 on my computer and project build initially complained about multiple StyleCop configurations, so I am uncertain if everything is in order.

@Sefriol
Copy link
Author

Sefriol commented Jun 14, 2023

@microsoft-github-policy-service agree

Previously nested and array JSON was converted to flat dictionary.
@Sefriol Sefriol force-pushed the feature/json-template branch from 3757ec8 to 79bf1f5 Compare June 14, 2023 07:05
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