Skip to content

feat(dashboard): Add overview dashboard #11416

Merged
billyvg merged 7 commits into
masterfrom
feat/dashboard/static-overview-dashboard
Jan 15, 2019
Merged

feat(dashboard): Add overview dashboard #11416
billyvg merged 7 commits into
masterfrom
feat/dashboard/static-overview-dashboard

Conversation

@billyvg

@billyvg billyvg commented Jan 8, 2019

Copy link
Copy Markdown
Member

@billyvg billyvg force-pushed the feat/dashboard/static-overview-dashboard branch 7 times, most recently from 47d6d7b to 85389df Compare January 10, 2019 19:06
@billyvg billyvg requested review from a team and lynnagara January 10, 2019 19:07
@billyvg

billyvg commented Jan 10, 2019

Copy link
Copy Markdown
Member Author

Please review the last commit

@billyvg billyvg force-pushed the feat/dashboard/static-overview-dashboard branch 2 times, most recently from e51366c to 53769af Compare January 11, 2019 00:51
@billyvg billyvg changed the title [WIP] feat(dashboard): Add overview dashboard feat(dashboard): Add overview dashboard Jan 11, 2019
@billyvg billyvg force-pushed the feat/dashboard/static-overview-dashboard branch from 53769af to 3622491 Compare January 11, 2019 21:27
Comment thread src/sentry/static/sentry/app/views/organizationDashboard/discoveryQuery.jsx Outdated
Comment thread src/sentry/static/sentry/app/views/organizationDashboard/discoveryQuery.jsx Outdated
@billyvg billyvg force-pushed the feat/dashboard/static-overview-dashboard branch from 3622491 to fab1b9e Compare January 11, 2019 22:20
@billyvg billyvg requested a review from a team January 14, 2019 20:38
@lynnagara

Copy link
Copy Markdown
Member

This looks generally good to me - the only thing I'd ask is if we can pull out the "system queries" stuff from this PR since it's a bit confusing and isn't being used anywhere

@billyvg

billyvg commented Jan 14, 2019

Copy link
Copy Markdown
Member Author

This looks generally good to me - the only thing I'd ask is if we can pull out the "system queries" stuff from this PR since it's a bit confusing and isn't being used anywhere

Removed #11494

@lynnagara lynnagara left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

💯

Comment thread src/sentry/static/sentry/app/views/organizationDashboard/discoverQuery.jsx Outdated
@billyvg billyvg force-pushed the feat/dashboard/static-overview-dashboard branch from 2c74583 to 4ac1ae4 Compare January 14, 2019 23:50
@billyvg billyvg merged commit 896a201 into master Jan 15, 2019
@billyvg billyvg deleted the feat/dashboard/static-overview-dashboard branch January 15, 2019 00:39
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants