Partial Fix Issue #1226 .
We are not yet allowing user to supply 'ReportAs' for counters. We are evaluating the potential billing impact if non standard names are used.
For significant contributions please make sure you have completed the following items:
[ ] Changes in public surface reviewed
[ ] Design discussion issue #
[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue.
[ ] The PR will trigger build, unit tests, and functional tests automatically. If your PR was submitted from fork - mention one of committers to initiate the build for you.
If you want to to re-run the build/tests, the easiest way is to simply Close and Re-Open this same PR. (Just click 'close pull request' followed by 'open pull request' buttons at the bottom of the PR)
Partial Fix Issue #1226 . We are not yet allowing user to supply 'ReportAs' for counters. We are evaluating the potential billing impact if non standard names are used.
follow up to https://github.com/microsoft/ApplicationInsights-dotnet-server/pull/1224
For significant contributions please make sure you have completed the following items:
[ ] Changes in public surface reviewed
[ ] Design discussion issue #
[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue.
[ ] The PR will trigger build, unit tests, and functional tests automatically. If your PR was submitted from fork - mention one of committers to initiate the build for you. If you want to to re-run the build/tests, the easiest way is to simply Close and Re-Open this same PR. (Just click 'close pull request' followed by 'open pull request' buttons at the bottom of the PR)
Please follow [these] (https://github.com/Microsoft/ApplicationInsights-dotnet-server/blob/develop/CONTRIBUTING.md) instructions to build and test locally.