You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
I was digging in the code of the portal to find the source of an issue and I noticed that
The code is very sparsely commented,
We are missing a lot of execution logs that could be helpful to follow the actions on the portal and get some information about queries done to APIs and cloud providers to know more about the execution context.
Describe the solution you'd like
For 1., add comments to all methods and document the code inside methods when necessary, depending on complexity.
For 2., add execution logs and debug information for context, e.g. in AzureIoTHub.Portal.Infrastructure.Services.LoRaWanManagementService.ExecuteLoRaDeviceMessage, I don't know the query's target, the device Id, the URI of the tarfet API, ... which would provide more insight on the internal behavior of the portal without having to setup a local instance for debugging and bugfixes.
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem? Please describe.
I was digging in the code of the portal to find the source of an issue and I noticed that
Describe the solution you'd like
For 1., add comments to all methods and document the code inside methods when necessary, depending on complexity.
For 2., add execution logs and debug information for context, e.g. in
AzureIoTHub.Portal.Infrastructure.Services.LoRaWanManagementService.ExecuteLoRaDeviceMessage
, I don't know the query's target, the device Id, the URI of the tarfet API, ... which would provide more insight on the internal behavior of the portal without having to setup a local instance for debugging and bugfixes.Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: