checkly / public-roadmap

Checkly public roadmap. All planned features, updates and tweaks.
https://checklyhq.com
37 stars 7 forks source link

Increase the prominence of API check response time failure reasons on results page and Slack alerts #266

Open blinkfront opened 1 year ago

blinkfront commented 1 year ago

Is your feature request related to a problem? Please describe. When an API check fails solely because it has exceeded the configured response time, it is not immediately clear from the results page or the Slack alert why the check has failed. The response time being the reason for failure is not given much prominence.

Screenshot 2022-10-26 at 10 39 30

Currently on the results page for a failing check, the only apparent indication that a the time taken has caused a failures is a small red underline beneath the time on the upper right side of the page.

On Slack alerts, the only potential indication is the entry for response time, which is always displayed anyway so your attention isn't drawn to it as the failure reason.

Describe the solution you'd like Increase the visual prominence of the reason for failure on the results page where a timeout has caused an otherwise successful request to fail. Perhaps adding a ❌ near the response time, in a similar way to how assertion failures are highlighted.

Consider whether the display of the response time in the Slack alerting could be augmented to draw attention to it, perhaps with the addition of a ❌ symbol.

Additional context Attached is a (partially redacted) screenshot showing what we were presented with on a check which failed due to response time.

As an aside, using just a red/yellow underline to indicate failed/degraded may not meet accessibility requirements, as it is using only colour to convey meaning.

tnolet commented 1 year ago

Hey @blinkfront that is some great feedback. Let me ping our designer @orcunilbeyli and @jenya-II

jenya-II commented 1 year ago

i created task for Design team, will tackle it a bit later, looks like quick and easy win for us thank you @blinkfront