Why is an error response lighting the healthy indicator?
NoraChen1192 · 5 Apr 2025, 14:34 UTC
15 replies
Can you compare the raw successful and failed responses against the documentation for the installed version? I can't infer the structure from the word container, and guessing which element means healthy could just move the mistake somewhere less obvious.
7pointsThe developer captured both and confirmed that the failed response is non-empty; our code converts the whole object to a Boolean.
11pointsThat explains the misleading green, although the replacement still needs to follow the documented success and data fields. A response being present and its status data being valid are two separate conditions.
17pointsPlease show a failed read as unavailable or unknown, with wording your operators recognise. Leaving the old healthy value looking current makes me think nothing has changed, even if a small error message appears somewhere else.
11pointsGood point. Preserve the last known value only if its age and stale state are clear; don't let a failed fresh request lend it the appearance of current evidence.
7pointsThe draft now parses success before using the payload and marks failed reads unavailable; it also labels a retained last reading as stale with its observation time.
17pointsInclude malformed and unexpected responses in the tests as well as the documented failure example. Your display should not fall back to a plausible healthy value when parsing cannot establish a valid result.
7pointsAnd use text as well as colour; green and grey alone won't explain this reliably
2pointsWe added Current, Stale and Unavailable labels beside the value; the error explanation is accessible from the same area rather than a separate log page.
22pointsDoes unavailable mean the robot stopped, or only that your screen can't read it?
14pointsThat distinction belongs in the label. The failed read establishes that the dashboard lacks current information; it does not by itself establish the equipment's physical state or whether it is operating.
13pointsWe changed it to 'Current status unavailable' and clarified the connection error underneath. The earlier short label was too easy to read as an equipment diagnosis.
22pointsThat wording is much clearer from the operator side. Try it with a saved normal reading followed by a failure, because that's when the reassuring old value is most likely to catch the eye first.
8pointsThe developer verified successful, failed, malformed and stale sequences against the installed interface documentation, and operators reviewed the transitions. The false healthy indication is gone in those checks. We have deployed the corrected read-only display.
25pointsKeep the installed SDK version with those captured examples. They're useful regression evidence when the environment changes again, and far less ambiguous than a screenshot of a green indicator.
6pointsDiscussion closed
This discussion is closed to new replies after six months without activity. Last activity: .