Azure / pcs-remote-monitoring-webui

Azure IoT PCS Remote Monitoring Web UI
MIT License
54 stars 72 forks source link

Bug fix for Alerts on device details page #1202

Closed sushilraje closed 5 years ago

sushilraje commented 5 years ago

Type of change?

Description, Context, Motivation

The alerts section on the device details page flyouts was showing ALL alerts for all the device(s) rather than showing alerts for the particular device only. The issue was traced to the POST calls to Telemetry service. The call body contained JSON attributes in lower case, but JAVA JsonProperty mapping expected Pascal case attributes. To make the calls consistent with mappings (QueryApiModel) the attributes,

  1. Devices
  2. From
  3. To
  4. Limit
  5. Order

are converted into Pascal case. This does not break the dotnet version.

Checklist:


This change is Reviewable