Skip to content

Conversation

hieptl
Copy link
Collaborator

@hieptl hieptl commented Sep 26, 2025

  • This change is worth documenting at https://docs.all-hands.dev/
  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

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

Task Overview:

  • The use-ws-error-message.ts file must be migrated to use Zustand for state management.

Acceptance Criteria:

  • The use-ws-error-message.ts file is successfully refactored to utilize Zustand.
  • All existing functionality remains intact, with no regressions or broken features.

Summarize what the PR does, explaining any non-trivial design decisions.

The PR updates the code to migrate use-ws-error-message.ts to Zustand.


Link of any specific issues this addresses:


To run this PR locally, use the following command:

GUI with Docker:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:a4139ed-nikolaik   --name openhands-app-a4139ed   docker.all-hands.dev/all-hands-ai/openhands:a4139ed

CLI with uvx:

uvx --python 3.12 --from git+https://github.com/All-Hands-AI/OpenHands@hieptl/all-3669 openhands

@hieptl hieptl requested a review from amanape as a code owner September 26, 2025 15:07
Copy link
Contributor

Coverage report

This PR does not seem to contain any modification to coverable code.

Copy link

openhands-ai bot commented Sep 30, 2025

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • Run Python Tests

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #11147 at branch `hieptl/all-3669`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

@hieptl hieptl merged commit 2eb3a9e into main Sep 30, 2025
35 of 38 checks passed
@hieptl hieptl deleted the hieptl/all-3669 branch September 30, 2025 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants