additional dashpages changes
This includes a merge of main into this branch at the end to resolve outstanding conflicts. I know the diff is large. However, you can ignore all but the last/3rd/4th/5th/6th/7th commits, which are all small and isolated to fix the issue in their commit names.
- consistently sort dashpages in the overall dashpages page
- rename name to display name, add id parameter to DashpageDefinition and use id where appropriate
- rename Dashpages to Highlights, update the query parameter and path parameter from dashpage/dashpages to highlight/highlights
Microsoft Reviewers: Open in CodeFlow
@drewnoakes
Seems the test PlotlyChartTests.Render_NoInstrument_NoPlotlyInvocations is failing due to differences in HTML.
Could you rebase on the branch you're merging to? Looking through commits makes reviewing harder. And you automatically include a bunch of unrelated people to review:
Could you rebase on the branch you're merging to? Looking through commits makes reviewing harder. And you automatically include a bunch of unrelated people to review:
@adamint for the other merges from main I've just pushed directly to the feature branch, after checking that everything builds/passes locally.
Added a warning comment, updated APIs -> highlights
/azp run
Azure Pipelines successfully started running 1 pipeline(s).
/azp run
Azure Pipelines successfully started running 1 pipeline(s).