Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

Update pet-shop.md #133

Closed
wants to merge 1 commit into from
Closed

Update pet-shop.md #133

wants to merge 1 commit into from

Conversation

abancu
Copy link

@abancu abancu commented Jul 24, 2017

I also want to say that I personally used a virtual machine in Vagrant, and I opened my 8545 port into firewall so I can reach blockchain by ip instead of localhost. For those who has issues with this testrpc has to be started with optional parameters --hostname (I only succeeded to bind it to 0.0.0.0 to work properly from chrome with metamask); Also if there is any other extension available, rather than metamask would be awesome as I had to reinstall extension to reset parameters.

@OnlyOneJMJQ
Copy link
Contributor

@abancu Thanks the function name is now fixed! Didn't merge only because it was fixed in a big release update we put out earlier today.

For the Vagrant woes, I've opened an issue to investigate: #141

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants