Skip to content

Commit

Permalink
ui: fix Overview screen in OSS builds
Browse files Browse the repository at this point in the history
Previously, when using OSS builds (created with `make buildoss`), when
you loading the DB Console in your browser, you'd get "Page Not Found".
The route for the overview page was missing the leading '/'.  This bug
appears to have been introduced in
722c932.

Release note (admin ui change): This fixes a bug where users of
the OSS builds of CockroachDB would see "Page Not Found" when loading
the Console.
  • Loading branch information
jclulow authored and koorosh committed Apr 26, 2021
1 parent 5e4b633 commit 153f65b
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions pkg/ui/src/routes/visualization.tsx
Expand Up @@ -31,11 +31,13 @@ class NodesWrapper extends React.Component<{}, {}> {

export default function createClusterOverviewRoutes(): JSX.Element {
return (
<Route path="overview" component={ ClusterOverview } >
<Switch>
<Route exact path="/" component={() => <Redirect to="list" />}/>
<Route path="list" component={ NodesWrapper } />
</Switch>
<Route path="/overview">
<ClusterOverview>
<Switch>
<Redirect exact from="/overview" to="/overview/list" />
<Route path="/overview/list" component={ NodesWrapper } />
</Switch>
</ClusterOverview>
</Route>
);
}

0 comments on commit 153f65b

Please sign in to comment.