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

Add BotTelemetry instrumentation and utilities #662

Merged
merged 38 commits into from
Dec 5, 2018
Merged
Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
6edd287
Initial broken attempt
daveta Nov 21, 2018
aa0b1e6
Move endDialog App Insights logging where it should be..
daveta Nov 26, 2018
73f780f
Placeholder for TelemetryInitializer
daveta Nov 26, 2018
449649e
placeholder initializer
daveta Nov 26, 2018
00de015
Forgot this
daveta Nov 26, 2018
7279ab7
Merge branch 'master' of github.com:Microsoft/botbuilder-js into dave…
benbrown Nov 26, 2018
20d2765
flesh out telemetry client
benbrown Nov 26, 2018
b73ec81
create fully functional application insights plugin
benbrown Nov 27, 2018
994514c
Merge branch 'master' of github.com:Microsoft/botbuilder-js into dave…
benbrown Nov 27, 2018
f6218f8
Merge branch 'daveta-appsight-js' of github.com:Microsoft/botbuilder-…
benbrown Nov 27, 2018
b75bf42
Update telemetry client to use botbuilder-core directly
benbrown Nov 28, 2018
1ac1c91
Add instanceId to each waterfallDialog
benbrown Nov 29, 2018
703029a
Expose options to developers can tweak behavior
benbrown Nov 29, 2018
980d3b7
Add some comments
benbrown Nov 29, 2018
d67f628
add more sample code
benbrown Nov 29, 2018
cc5b31e
Unit tests for waterfall dialog implementation
benbrown Nov 29, 2018
aa8f256
protect against missing step name
benbrown Nov 29, 2018
8118995
Update with improved typescript definitions
benbrown Nov 30, 2018
7d28169
update syntax for private variables
benbrown Dec 3, 2018
caa97e6
update typescript definitions and some style issues
benbrown Dec 3, 2018
52f81c6
Merge branch 'master' into daveta-appsight-js
benbrown Dec 3, 2018
95137ea
Add new package to main readme
benbrown Dec 3, 2018
128cdfe
update readme
benbrown Dec 3, 2018
298b469
readme
benbrown Dec 3, 2018
bc44bf0
crosslink appinsights docs
benbrown Dec 3, 2018
b29bcde
crosslink appinsights docs
benbrown Dec 3, 2018
972fccf
Add handling of telemetryClient to dialogSet and component Dialog
benbrown Dec 4, 2018
dcb2de6
add test for null setting
benbrown Dec 4, 2018
6f83045
update readme
benbrown Dec 4, 2018
787be9b
Merge branch 'master' of github.com:Microsoft/botbuilder-js into dave…
benbrown Dec 4, 2018
251384d
rollback package lock to last workign state
benbrown Dec 4, 2018
c86e582
updated package
benbrown Dec 4, 2018
4904059
update package fies
benbrown Dec 4, 2018
4aabfbe
packge flies
benbrown Dec 4, 2018
16d6acd
reset package locks
benbrown Dec 4, 2018
a811a18
run tools install
benbrown Dec 4, 2018
af83bb0
Merge branch 'master' into daveta-appsight-js
benbrown Dec 4, 2018
f88e8f0
Merge branch 'master' into daveta-appsight-js
stevengum Dec 5, 2018
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions libraries/botbuilder-applicationinsights/.gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/**/node_modules
/**/.vscode
/**/lib
coverage
.nyc_output
4 changes: 4 additions & 0 deletions libraries/botbuilder-applicationinsights/.npmignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
/**/node_modules
/**/.vscode
coverage
.nyc_output
19 changes: 19 additions & 0 deletions libraries/botbuilder-applicationinsights/.nycrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
{
"extension": [
".js"
],
"include": [
"lib/**/*.js"
],
"exclude": [
"**/node_modules/**",
"**/tests/**",
"**/coverage/**",
"**/*.d.ts"
],
"reporter": [
"html"
],
"all": true,
"cache": true
}
41 changes: 41 additions & 0 deletions libraries/botbuilder-applicationinsights/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
# Bot Builder Application Insights

Application Insights extensions for Microsoft BotBuilder.

- [Installing](#installing)
- [Basic Use](#use)
- [Documentation](https://docs.microsoft.com/en-us/azure/bot-service/bot-service-overview-introduction?view=azure-bot-service-4.0)
- [Class Reference](https://docs.microsoft.com/en-us/javascript/api/botbuilder-azure/)
- [GitHub Repo](https://github.com/Microsoft/botbuilder-js)
- [Report Issues](https://github.com/Microsoft/botbuilder-js/issues)

## Installing
To add the latest version of this package to your bot:

```bash
npm install --save botbuilder-applicationinsights
```

#### Use the Daily Build

To get access to the daily builds of this library, configure npm to use the MyGet feed before installing.

```bash
npm config set registry https://botbuilder.myget.org/F/botbuilder-v4-js-daily/npm/
```

To reset the registry in order to get the latest published version, run:
```bash
npm config set registry https://registry.npmjs.org/
```

## What's Included

This module contains interfaces to use common Application Insights services to back Bot Builder's metrics and reporting needs.

## Use

Import the module into your app.
```javascript
TODO
```
Loading