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

Completed assessment #1

Closed
wants to merge 4 commits into from
Closed

Completed assessment #1

wants to merge 4 commits into from

Conversation

danielcolinjames
Copy link

I'm not sure if I did everything the correct way, because this is all pretty new to me, but all the functionality seems to work :)

I only gave it a quick styling pass after I got everything working—I would normally spend way more time on the design. I used Tailwind because it's the styling syntax I'm most familiar with and it lets me move super quickly. If I took more time I would probably also refactor the codebase to make it more componentized.

I ran into some issues as I went, which I documented in case it's helpful:

  • I was getting a MetaMask error when trying to add localhost:8545, because I didn't realize it was something the hardhat command would create (I think that's what happened?) and that I didn't have to manually add it
  • Using Brave browser was making it not recognize my connected wallets through MetaMask, and CurrentAddressContext[0] was showing only Vic's wallet address, not my connected wallet. Apparently it's a Brave thing: https://ethereum.stackexchange.com/a/94856, so I used Chrome
  • I went down a bit of an ENS rabbit hole trying to get it to show an ENS name if there was one set:
  • Error when trying to send a transaction (fixed by adding chainid: 1337 to hardhat config and restarting hardhat):
eth_sendRawTransaction
Incompatible EIP155-based V 2710 and chain id 31337. See the second parameter of the Transaction constructor to set the chain id.
If you are using MetaMask, you can learn how to fix this error here: https://hardhat.org/metamask-issue

For the "stealing TST" part, I actually started writing an email to you asking for help and explaining what I'd tried so far, but the act of writing the email helped me get unstuck and figure it out.

Anyway, this was a great introduction to some basic web3 stuff! I learned a lot while going through it—thanks for putting it together :)

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.

2 participants