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.

Sign up for GitHub

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

Jump to bottom

fixed #11 - Uncaught (in promise) ReferenceError: Web3 is not defined #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fixed #11 - Uncaught (in promise) ReferenceError: Web3 is not defined #12

wants to merge 2 commits into from

Conversation

jdohm
Copy link

@jdohm jdohm commented Apr 17, 2021

This issue is due to the fact that metamask no longer injects web3 by default

set default account to use on web3
this resolves the following issue:
Uncaught (in promise) Error: invalid address
at inputAddressFormatter (formatters.js:274)

package.json

  • added web3 dependency

src/app.js

  • added default account

src/index.html

  • added web3 dependency

@jdohm
fixed dappuniversity#11 - Uncaught (in promise) ReferenceError: Web3 …
5f2a159 
…is not defined

This issue is due to the fact that metamask no longer injects web3 by default

package.json
- added web3 dependency

src/index.html
- added web3 dependency
@jdohm
set default account to use on web3
21f522c 
this resolves the following issue:
  Uncaught (in promise) Error: invalid address
    at inputAddressFormatter (formatters.js:274)
@jdohm jdohm mentioned this pull request Apr 17, 2021
Open
@krishprasadar
Copy link

Thanks for initiating this PR! Saved a lot of time for me :) Your change works!

@streetsmatter
Copy link

Thank You!!!

@david-randoll
Copy link

Thank you!!

@bloolizard
Copy link

Thanks! Works!

@johnwalking
Copy link

it helps a lot, thanks

@boyerst
Copy link

boyerst commented Oct 6, 2021

Thanks @jdohm! Worked like a charm

@ghost
Copy link

ghost commented Nov 22, 2021

Great fix - thanks for this

@montrealks
Copy link

Thank you!

@wenjoy
Copy link

wenjoy commented Apr 5, 2022

Good job. It's killed me

@mrcoulter45
Copy link

Thanks! I vote this gets merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anilnabin25 anilnabin25 anilnabin25 approved these changes

@ByWaleed ByWaleed ByWaleed approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.