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

Add POA.network #1365

Closed
wants to merge 1 commit into from
Closed

Add POA.network #1365

wants to merge 1 commit into from

Conversation

rstormsf
Copy link
Contributor

@rstormsf rstormsf commented Mar 22, 2018

Closes #1364

Description

Adding https://core.poa.network Chain id: 99, was already supported in MEW, mycrypto
EIP155 protected network

I filed an issue #1363 because I don't believe that other networks should suffer if Metamask team won't add some ETH-like network, doesn't mean the network doesn't work with metamask
MetaMask/metamask-extension#2815

Also, there are proposals on how it might actually work without having metamask to make political decisions.

MetaMask/metamask-extension#3604

But any network works with metamask by using Custom RPC port. The same way it did with MEW and mycrypto.com
I don't see why it shouldn't work in beta.mycrypto.com

@rstormsf rstormsf mentioned this pull request Mar 22, 2018
@rstormsf
Copy link
Contributor Author

any idea why travis failed?
I also see that previous PR also failed:
https://travis-ci.org/MyCryptoHQ/MyCrypto/builds/356708499

What should I do?

@dternyak
Copy link
Contributor

@rstormsf Looks like you had a failed test!
screen shot 2018-03-22 at 12 59 21 am

https://travis-ci.org/MyCryptoHQ/MyCrypto/builds/356708499

We're actually already working on integrating POA in the PR found here: https://github.com/MyCryptoHQ/MyCrypto/pull/1259/files. Perhaps you could help us validate the implementation there? :)

@rstormsf
Copy link
Contributor Author

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