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

[Uptime] Remove map from Uptime details page #86485

Closed
andrewvc opened this issue Dec 18, 2020 · 5 comments · Fixed by #96517
Closed

[Uptime] Remove map from Uptime details page #86485

andrewvc opened this issue Dec 18, 2020 · 5 comments · Fixed by #96517
Labels
enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan-ok issue has passed test plan v7.13.0

Comments

@andrewvc
Copy link
Contributor

The map view of monitor locations in the Uptime app doesn't provide much value vs. the list with availability and recency. I propose we consider removing it unless someone can name a use case that still makes it useful. The only value it adds is showing exact locations on a map, which is useful if the location names are not descriptive enough. I think this is probably pretty rare.

CC @drewpost @katrin-freihofner @paulb-elastic

@andrewvc andrewvc added enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability [zube]: Investigate labels Dec 18, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@paulb-elastic
Copy link
Contributor

Is it causing us an issue, and so we want to remove it? A use case I can imagine would be to see failures from clusters in specific regions, e.g. is the website only down to visitors from Europe? Wouldn't the map view make that easier to determine?

@andrewvc
Copy link
Contributor Author

Yes, it's a burden to maintain, and a fairly heavy dependency. I don't doubt that it has some utility, but it hardly seems worth it to maintain.

The default view is a list of monitor locations by name, which in the example of 'europe' would be just as easy to determine, since you'd have a testing location named central-europe or similar.

@paulb-elastic
Copy link
Contributor

As discussed here this will be removed

@justinkambic
Copy link
Contributor

Ran locally, map is indeed gone. On a personal note, I'm happy to see this dependency go away as it was a pain to maintain and made our platform migration 5x as difficult as it could've otherwise been.

@justinkambic justinkambic added test-plan-ok issue has passed test plan and removed test-plan labels Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan-ok issue has passed test plan v7.13.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants