Skip to content

Commit

Permalink
Fix dashboard building check
Browse files Browse the repository at this point in the history
  • Loading branch information
AdityaHegde committed Jan 14, 2025
1 parent fa0b7d3 commit f36ecc1
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,13 @@
$explore.error?.response?.status === 404;
// TODO: should these be checking metricsView or explore?
$: isDashboardReconcilingForFirstTime =
$explore?.data?.metricsView?.metricsView?.state?.validSpec === null &&
!$explore?.data?.metricsView?.metricsView?.state?.validSpec &&
!$explore?.data?.metricsView?.meta?.reconcileError;
// We check for metricsView.state.validSpec instead of meta.reconcileError. validSpec persists
// from previous valid dashboards, allowing display even when the current dashboard spec is invalid
// and a meta.reconcileError exists.
$: isDashboardErrored =
$explore?.data?.metricsView?.metricsView?.state?.validSpec === null &&
!$explore?.data?.metricsView?.metricsView?.state?.validSpec &&
!!$explore?.data?.metricsView?.meta?.reconcileError;
$: metricsViewName = $explore.data?.metricsView?.meta?.name?.name;
Expand Down

0 comments on commit f36ecc1

Please sign in to comment.