-
Notifications
You must be signed in to change notification settings - Fork 0
migrate from remix to react-router 7 and update everything #2
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR migrates the application from Remix to React Router 7, updating all dependencies and changing form handling from remix-validated-form to @rvf/react-router. The migration modernizes the codebase with new routing patterns, updated dependencies, and improved form validation.
Key Changes:
- Complete migration from Remix to React Router 7 framework
- Form validation library change from
remix-validated-formto@rvf/react-router - Tailwind CSS upgrade to v4 with new plugin system and CSS structure
Reviewed Changes
Copilot reviewed 70 out of 73 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| vite.config.ts | Updated build configuration for React Router |
| package.json | Dependency updates for React Router 7 ecosystem |
| app/routes/*.tsx | Route components updated for new framework |
| app/components/*.tsx | Form components migrated to new validation library |
| tailwind.css | Restructured for Tailwind v4 plugin system |
Comments suppressed due to low confidence (2)
app/components/BuyItemCard/index.tsx:127
- The
formIdprop is being passed but not defined in scope. This should be removed as the form prop is handled by the FormApi.
{b.cost} CR - {b.name} ({'I'.repeat(b.tier)})
app/components/RebelClassManager/index.tsx:128
- Invalid CSS class
cursor-default!- the exclamation mark should not have a space. It should becursor-default!.
className="label cursor-default! gap-2 flex py-1"
No description provided.