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

Update api-filtering-sampling.md #42172

Merged
merged 4 commits into from
Nov 8, 2019
Merged

Update api-filtering-sampling.md #42172

merged 4 commits into from
Nov 8, 2019

Conversation

ramthi
Copy link
Contributor

@ramthi ramthi commented Nov 6, 2019

Adding Javascript section for filtering through ITelemetryInitializers.
Updating confusing headings (removed ITelemetryProcessor) from filtering section heading as filtering is done via initializers in JS!

Adding Javascript section for filtering through ITelemetryInitializers.
Updating confusing headings (removed ITelemetryProcessor) from filtering section heading as filtering is done via initializers in JS!
@PRMerger7
Copy link
Contributor

@ramthi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Nov 6, 2019
ramthi and others added 2 commits November 7, 2019 13:27
Co-Authored-By: Mark Wolff <mrw00010@gmail.com>
Co-Authored-By: Mark Wolff <mrw00010@gmail.com>
@PRMerger13
Copy link
Contributor

@ramthi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

1 similar comment
@PRMerger13
Copy link
Contributor

@ramthi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jborsecnik
Copy link
Contributor

@mrbullwinkle, okay to merge? Thanks.

@mrbullwinkle
Copy link
Contributor

#sign-off

@PRMerger12
Copy link
Contributor

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver ktoliver merged commit 9e083cb into MicrosoftDocs:master Nov 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants