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

Fix node modules and minor issues #1

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

Conversation

kiknaio
Copy link

@kiknaio kiknaio commented Nov 8, 2022

Hi Dan,

I was playing around with this repo and figured out that there were several issues and fixed those.
Fixes include:

  • Removing homepage from package.json. With the homepage value, when you run locally it redirects to the incorrect path http://localhost:3000/danfinlay/LazyConnec
  • Fix conflict with dependencies
  • Fix of eth_requestAccounts
  • Instead of loading a dummy App.js component, now <LazyConnect /> is rendering
  • Updated React version

@kiknaio
Copy link
Author

kiknaio commented Nov 10, 2022

I think I can add some improvements to the repo if it's still maintained

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.

None yet

1 participant