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
Migrating to react 18 #105
Comments
nope, dead repo |
Hey, I forked the repo and I'm trying to migrate to react 18. Did get it working but I'm still getting some legacy warnings here and there. Hopefully I'll get it working. |
let me know if I can help Would be nice if we could merge with original repo somehow |
@mariaSlimani I solved the legacy warning. I replaced the deprecated context with React.useContext. Works great for my application React:https://github.com/JacobDel/avataaars |
Hey @JacobDel ! first, thanks for taking the initiative! (if you saw the mess my commits were, you'd have figured I was having trouble with it XD). My app works normally when i install my version using npm install mariaSlimani/avataaars (if we were to ignore the warnings that is). |
I forgot to build it to the dest folder (was testing it with src files). Does it work now? |
Now when i install it I get : |
I found the issue, I'm working on it |
should be fixed @mariaSlimani |
Is it fixed now @mariaSlimani ? |
Is there any plans to migrate to react 18?
The text was updated successfully, but these errors were encountered: