-
-
Notifications
You must be signed in to change notification settings - Fork 584
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
[$35] Support React 16 + 17 #534
Comments
/bounty $35 |
💎 $35 bounty created by aidenybai
|
/attempt #534 this'd just involve supporting it in https://github.com/aidenybai/million/tree/main/packages/react & react-server, right? Options |
Yes |
Thanks, on it! |
How do I validate if my fixes are correct? |
@rishi-raj-jain create a new vite project. use react 17 + react-dom 17. check if it works. then try with react 18. if it works on both, you can get the bounty |
On it, thanks for the direction! |
Works fine with v18, breaks with v16 and v17 for react-dom errors. |
Ohkay, the only problem is of react-dom/client and shall be using react-dom in react < 18. Working on that. |
I have started working on that issues. I have steuped vite with react 18 and is also supported in react 16/17. Kindly make sure Is it right? then I can share zip file on that |
@aidenybai Please guide me Am i right? |
💡 @rishi-raj-jain submitted a pull request that claims the bounty. You can visit your org dashboard to reward. |
Hi @shubhamsingh259. You are right. You are to confirm that Million.js works in your vite app without errors when you use v16, v17 and v18 of React and React DOM. Once that is done, open a PR and claim the bounty. |
ok thanks |
🎉🎈 @rishi-raj-jain has been awarded $35! 🎈🎊 |
I have created vite with react18 project. I have also intsalled million.js in vite+react project. |
Please check this repository and kindly guide me. Am I right?. |
Congratulations!
…On Fri, 18 Aug 2023, 10:18 algora-pbc[bot], ***@***.***> wrote:
🎉🎈 @rishi-raj-jain <https://github.com/rishi-raj-jain> has been awarded
*$35*! 🎈🎊
—
Reply to this email directly, view it on GitHub
<#534 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOEEFHMJZGYYI7G6ZCMQ7YDXV3XZ7ANCNFSM6AAAAAA3TDDGFI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
For anyone else stumbling here–see #1070 (comment) |
Describe the feature
Currently Million.js supports React 18. It would be nice to support React 16/17 builds.
Additional information
The text was updated successfully, but these errors were encountered: