Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alexwykoff/add sentry link #55

Closed
wants to merge 3 commits into from
Closed

Conversation

alexwykoff
Copy link
Contributor

Fixes: #53

@@ -56,6 +57,7 @@ const Main = ({ config }) => {
<Route path='/metadata' component={Metadata} />
<Route path='/signaling' component={Signaling} />
<Route path='/system' component={System} />
<Route path='/sentry' component={Sentry} />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/logging

@@ -28,6 +28,7 @@ import Metadata from './panels/Metadata';
import Signaling from './panels/signal/Signaling';
import System from './panels/system/System';
import Registry from './panels/registry/Registry';
import Sentry from './panels/sentry/Sentry';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logger


//TODO(wykoff): add query tab, add commonly used info tab
return (
<Panel
Copy link
Member

@richburdon richburdon Nov 25, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove panel and content (don't need tabs).
Add a Toolbar with a link (see StatusBar.js)

      <Toolbar className={classes.root}>
        <div className={classes.left}>
          <Link className={classes.link} href={config.app.website} rel='noreferrer' target='_blank'>
            <PublicIcon />
          </Link>
        </div>
      </Toolbar>

@telackey telackey closed this Dec 10, 2020
@dmaretskyi dmaretskyi deleted the alexwykoff/add-sentry-link branch February 5, 2021 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a link to sentry
3 participants