Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(frontend): Migrate from Remix to React Router 7 #5304

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

amanape
Copy link
Member

@amanape amanape commented Nov 27, 2024

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

  • We are not utilizing data loaders

  • We are not using SSR

  • React Router v7 is the next iteration of React Router v6 and Remix v2

  • 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

  • Replace Remix dependencies with React Router
  • Update imports
  • Update scripts
  • Update vite config
  • Update how routes are handled (no longer file-based)

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:6962032-nikolaik   --name openhands-app-6962032   docker.all-hands.dev/all-hands-ai/openhands:6962032

@amanape amanape self-assigned this Nov 27, 2024
@amanape amanape marked this pull request as ready for review November 27, 2024 20:11
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.

1 participant