Skip to content

Enhance ILogger documentation with telemetry key setup - #5502

Closed
Glen Fu (gfugy) wants to merge 2 commits into
MicrosoftDocs:mainfrom
gfugy:patch-2
Closed

Glen Fu (gfugy) wants to merge 2 commits into
MicrosoftDocs:mainfrom
gfugy:patch-2

Conversation

@gfugy

Copy link
Copy Markdown
Contributor

Added instructions for updating the telemetry instrumentation key in Dataverse environment.

Added instructions for updating the telemetry instrumentation key in Dataverse environment.
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 9ee04a7:

⚠️ Validation status: warnings

File Status Preview URL Details
powerapps-docs/developer/data-platform/application-insights-ilogger.md ⚠️Warning Details

powerapps-docs/developer/data-platform/application-insights-ilogger.md

  • Line 26, Column 72: [Warning: disallowed-html-tag - See documentation] HTML tag 'Application' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Updated the command syntax for setting the telemetry instrumentation key.

@gfugy Glen Fu (gfugy) left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected markdown syntax

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 2383e31:

✅ Validation status: passed

File Status Preview URL Details
powerapps-docs/developer/data-platform/application-insights-ilogger.md ✅Succeeded

For more details, please refer to the build report.

@JimDaly

Copy link
Copy Markdown
Contributor

Glen Fu (@gfugy)
We need to make sure that the correct way to enable telemetry capture is documented. I'm not sure this is the right way.
Looping in some people internally on email thread.

@JimDaly

Copy link
Copy Markdown
Contributor

Glen Fu (@gfugy) In the email discussion we concluded that the steps described in this PR are not necessary. Instead people should only use the instructions found in Create an export package. This is the only correct way to set this.

As documented, the person setting this must have Administrator, Contributor, or Owner access to the Azure Application Insights resource.

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.

3 participants