OpenHands
OpenHands copied to clipboard
fix(frontend): Fix `actionHasObservationPair` function to correctly identify pairs
- [ ] 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. We would show actions + observations instead of replace actions with their obsevation
This was because some actions come with thoughts, which we want to render. But we didnt take into account actions that didnt include them, so we rendered those instead of replace them with thoughts
Summarize what the PR does, explaining any non-trivial design decisions.
Link of any specific issues this addresses:
To run this PR locally, use the following command:
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:437b471-nikolaik --name openhands-app-437b471 docker.all-hands.dev/all-hands-ai/openhands:437b471
Looks like there are a few issues preventing this PR from being merged!
- GitHub Actions are failing:
- Run Python Unit Tests
If you'd like me to help, just leave a comment, like
@OpenHands please fix the failing actions on PR #8661
Feel free to include any additional details that might help me get this PR into a better state.
You can manage your notification settings