Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

Fixing bug I found during AppLens 101 Workshop #736

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fixing bug I found during AppLens 101 Workshop #736

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
252b1f3
Update DynamicInsight.cs
tomrlake Sep 14, 2021
51bfb20
Merge pull request #1 from tomrlake/tomrlake-patch-1
tomrlake Sep 14, 2021
100f3a0
Merge branch 'main' into main
tomrlake Sep 23, 2021
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
4 changes: 2 additions & 2 deletions src/Diagnostics.ModelsAndUtils/Models/ResponseExtensions/DynamicInsight.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,9 @@ public static partial class ResponseInsightsExtension
/// public async static Task<![CDATA[<Response>]]> Run(DataProviders dp, OperationContext cxt, Response res)
/// {
/// var diagnosticData = new DiagnosticData {
/// Table = table //This is where you would execute kusto query
/// Table = table, //This is where you would execute kusto query
/// RenderingProperties = new TimeSeriesRendering()
/// }
/// };
///
/// DynamicInsight insight = new DynamicInsight(InsightStatus.Critical, "This insight will expand to show a graph", diagnosticData);
///
Expand Down