Skip to content

[PORT] Fixes for QnA / Recognizer Adaptive telemetry #960

@tomlm

Description

@tomlm

Port this change from botbuilder-dotnet/master branch:
microsoft/botbuilder-dotnet#3715

  • Adds UseTelemetry extension method for DiaogManager which adds IBotTelemetryClient to turn state for use by adaptive recognizers.
  • Adaptive recognizers will now use the IBotTelemetryClient on the turn state unless the TelemetryClient property on the recognizer has been set.
  • Adds LogPersonalInformation property to QnAMaker dialogs and LUIS / QnA adaptive recognizers. Adaptive LUIS / QnA will look in settings.telemetry.logPersonalInformation by default.
  • Ensures QnAMaker client is always passed a TelemetryClient if set, along with the LogPersonalInformation flag (Fixes #3472)
  • LUIS Adaptive recognizer: Ensure telemetry properties passed in can override default properties

@cwhitten @vishwacsena Once these changes have been approved, I can raise an issue for composer to detail the changes needed in the runtime for telemetry.

Changed projects

  • Microsoft.Bot.Builder.AI.QnA
  • Microsoft.Bot.Builder.Dialogs.Adaptive
  • Microsoft.Bot.Builder.Dialogs

[adaptive]

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions