SceneSolo: Minor fixes (#82289)

* SceneSolo: Minor fixes

* remove logging
pull/81185/head^2
Torkel Ödegaard 1 year ago committed by GitHub
parent cd09c36448
commit 8f36f905ee
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
  1. 1
      pkg/middleware/loggermw/logger.go
  2. 2
      public/app/features/dashboard-scene/sharing/ShareLinkTab.test.tsx
  3. 12
      public/app/features/dashboard-scene/sharing/ShareLinkTab.tsx
  4. 1
      public/app/features/dashboard-scene/sharing/SharePanelEmbedTab.tsx
  5. 4
      public/app/routes/routes.tsx

@ -74,6 +74,7 @@ func (l *loggerImpl) Middleware() web.Middleware {
duration := time.Since(start)
timeTaken := duration / time.Millisecond
ctx := contexthandler.FromContext(r.Context())
if ctx != nil && ctx.PerfmonTimer != nil {
ctx.PerfmonTimer.Observe(float64(timeTaken))
}

@ -80,7 +80,7 @@ describe('ShareLinkTab', () => {
await screen.findByRole('link', { name: selectors.pages.SharePanelModal.linkToRenderedImage })
).toHaveAttribute(
'href',
'http://dashboards.grafana.com/grafana/render/d-solo/dash-1?from=2019-02-11T13:00:00.000Z&to=2019-02-11T19:00:00.000Z&viewPanel=panel-12&width=1000&height=500&tz=Pacific%2FEaster'
'http://dashboards.grafana.com/grafana/render/d-solo/dash-1?from=2019-02-11T13:00:00.000Z&to=2019-02-11T19:00:00.000Z&panelId=panel-12&__feature.dashboardSceneSolo&width=1000&height=500&tz=Pacific%2FEaster'
);
});
});

@ -83,12 +83,20 @@ export class ShareLinkTab extends SceneObjectBase<ShareLinkTabState> {
shareUrl = await createShortLink(shareUrl);
}
// the image panel solo route uses panelId instead of viewPanel
let imageQueryParams = urlParamsUpdate;
if (panel) {
delete imageQueryParams.viewPanel;
imageQueryParams.panelId = panel.state.key;
// force solo route to use scenes
imageQueryParams['__feature.dashboardSceneSolo'] = true;
}
const imageUrl = getDashboardUrl({
uid: dashboard.state.uid,
currentQueryParams: location.search,
updateQuery: urlParamsUpdate,
updateQuery: { ...urlParamsUpdate, panelId: panel?.state.key },
absolute: true,
soloRoute: true,
render: true,
timeZone: getRenderTimeZone(timeRange.getTimeZone()),

@ -69,6 +69,7 @@ const getIframeBuilder =
params.set('panelId', panelId);
params.delete('editPanel');
params.delete('viewPanel');
params.set('__feature.dashboardSceneSolo', 'true');
const soloUrl = getDashboardUrl({
absolute: true,

@ -81,14 +81,14 @@ export function getAppRoutes(): RouteDescriptor[] {
),
},
{
path: '/d-solo/:uid/:slug',
path: '/d-solo/:uid/:slug?',
pageClass: 'dashboard-solo',
routeName: DashboardRoutes.Normal,
chromeless: true,
component: SafeDynamicImport(() =>
config.featureToggles.dashboardSceneSolo
? import(/* webpackChunkName: "SoloPanelPage" */ '../features/dashboard-scene/solo/SoloPanelPage')
: import(/* webpackChunkName: "SoloPanelPage" */ '../features/dashboard/containers/SoloPanelPage')
: import(/* webpackChunkName: "SoloPanelPageOld" */ '../features/dashboard/containers/SoloPanelPage')
),
},
// This route handles embedding of snapshot/scripted dashboard panels

Loading…
Cancel
Save