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.

Sign up for GitHub

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

Jump to bottom

Release note for JS FM v2 #1008

Open

zhiyuanliang-ms wants to merge 1 commit into main from zhiyuanliang/release-js-fm

Contributor

zhiyuanliang-ms commented Jan 14, 2025

No description provided.


          release js fm v2

2b4d285

zhiyuanliang-ms requested review from zhenlan, rossgrambo, jimmyca15, linglingye001 and Copilot

January 14, 2025 09:08

Copilot AI reviewed

View reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

releaseNotes/JavaScriptFeatureManagement.md:21

The phrase 'publish feature evaluation event to Application Insights' should be rephrased to 'publish feature evaluation events to Application Insights' for better clarity.

This hook can be used to publish telemetry. You can call `createTelemetryPublisher` API from `@microsoft/feature-management-applicationinsights-browser` and `@microsoft/feature-management-applicationinsights-node` packages to publish feature evaluation event to Application Insights.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

zhenlan Awaiting requested review from zhenlan

rossgrambo Awaiting requested review from rossgrambo

jimmyca15 Awaiting requested review from jimmyca15

linglingye001 Awaiting requested review from linglingye001

At least 1 approving review is required to merge this pull request.

Labels

None yet