Skip to content

Update ApplicationInsights quickstart#154

Open
dsschnau wants to merge 1 commit into
net-commons:masterfrom
dsschnau:patch-1
Open

Update ApplicationInsights quickstart#154
dsschnau wants to merge 1 commit into
net-commons:masterfrom
dsschnau:patch-1

Conversation

@dsschnau

@dsschnau dsschnau commented Oct 6, 2017

Copy link
Copy Markdown

Hi, I noticed that the quickstart for ApplicationInsights was 1. out of date and 2. not quite correct. I updated the versions to the latest AI (2.4.0) and added that you need to specify '240' at the end of the factoryAdapter config.

Hi, I noticed that the quickstart for ApplicationInsights was 1. out of date and 2. not quite correct. I updated the versions to the latest AI (2.4.0) and added that you need to specify '240' at the end of the `factoryAdapter` config.

@swisscoder swisscoder left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

corrected the package that needs to get installed from the package manager

Comment thread readme.md
There are different packages for each major Application Insights version. Install the correct package for your referenced Application Insights version. This example installs the adapter for Application Insights v2.4.0:

PM> Install-Package Microsoft.ApplicationInsights -Version 0.17.0
PM> Install-Package Microsoft.ApplicationInsights -Version 2.4.0

@swisscoder swisscoder Mar 7, 2018

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The correct way to install the package is actually this:
PM> Install-Package Common.Logging.ApplicationInsights240
(This also works together with the latest version of Microsoft.ApplicationInsights (2.5.1))

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants