-
Notifications
You must be signed in to change notification settings - Fork 418
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
Option on metrics histograms to show count #1265
Conversation
<value>Show Spec Only</value> | ||
<note>Spec means from the resource specification</note> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I edited ControlsStrings in the VS resource editor and it removed all the values with note.
@adamint Do you know what happened and how I can fix it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a bug in the Managed Resource Editor. Paging @melytc and @haileymck to investigate. cc @kvenkatrajan
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What should I do to fix this? I could revert all resx changes, hardcode the text, and create an issue to update in the future.
I have preview and int-preview on my dev machine. Does the bug impact both?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also happens in int preview
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, other than the regression in the resx. Approving on the basis that it would be reverted before merge.
e36812b
to
efb23ed
Compare
Fixes #965
Adds a switch to histogram charts to display the count of the instrument, e.g. the number of HTTP requests.
Demo: