OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

Provide a clearer error message when settings are missing midsession

Open raymyers opened this issue 11 months ago • 1 comments

End-user friendly description of the problem this fixes or functionality that this introduces

  • [ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Give a summary of what the PR does, explaining any non-trivial design decisions

The changes in #6101 made ConnectionRefusedError thrown in listen_socket.py display to the client. One of those scenarios is "Settings not found". This change gives those an optional "msg_id" for the translation and adds a user facing message that makes it more clear what action to take.

Another place this could be addressed is in frontend/src/hooks/query/use-settings.ts, but I'm not sure how to approach that and making these Websocket errors more first-class seemed like a good direction.


Link of any specific issues this addresses ALL-959

raymyers avatar Jan 08 '25 23:01 raymyers

New error

Screenshot from 2025-01-08 16-56-44

raymyers avatar Jan 08 '25 23:01 raymyers