-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Upgrade Remix template to React Router 7 #4900
Comments
💎 $250 bounty • RemotionSteps to solve:
Thank you for contributing to remotion-dev/remotion! |
Hey @JonnyBurger
|
@hunxjunedo: The Remotion team prefers to assign a single contributor to the issue rather than let anyone attempt it right away. We recommend waiting for a confirmation from a member before getting started. |
Me? |
No more attemptsThis issue has been assigned @hunxjunedo 🎉 Do not send pull requests for this issue, do not ask to be in the queue, do not bother the assignee. Follow the #bounties channel on the Remotion Discord to get notified of new bounties. |
React Router 7 is the successor to Remix.
We should upgrade our template:
https://github.com/remotion-dev/remotion/tree/main/packages/template-remix (Source of truth)
https://github.com/remotion-dev/template-remix (Standalone repo, which is copied from source of truth)
You can decide which one to work on, but the Remotion tests should pass on the template (validate-templates.test.ts)
Let's upgrade the template to use React Router 7 instead of Remix.
It should follow the new best React Router 7 practices, and keep the same functionality.
References in the code should be changed from "Remix" to "React Router 7".
💎 This issue has a bounty on it!
Read our contributing guidelines:
/bounty 250
The text was updated successfully, but these errors were encountered: