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

Deprecate endpoints according to updated spec #1270

Merged

Conversation

njgheorghita
Copy link
Contributor

@njgheorghita njgheorghita commented Mar 11, 2019

What was wrong?

RPC endpoints have changed according to new EIP-1474.

  • Deprecated web3.net.chainId()
  • Deprecated web3.eth.getCompilers()
  • Deprecated web3.eth.getTransactionFromBlock()
  • removed web3.eth.enableUnauditedFeatures()

Cute Animal Picture

image

@njgheorghita njgheorghita force-pushed the refactor-jsonrpc-endpoints branch 2 times, most recently from b2fb008 to 8bcd127 Compare March 11, 2019 14:20
@pipermerriam
Copy link
Member

pipermerriam commented Mar 11, 2019

It doesn't appear these have been properly deprecated in v4. We'll need to do that asap so that users won't be surprised by their disappearance.

@njgheorghita njgheorghita force-pushed the refactor-jsonrpc-endpoints branch 7 times, most recently from 69d59b2 to 98cf8d3 Compare March 11, 2019 15:38
@njgheorghita njgheorghita changed the title Clean up rpc endpoints according to updated spec Deprecate endpoints according to updated spec Mar 11, 2019
@njgheorghita njgheorghita force-pushed the refactor-jsonrpc-endpoints branch 4 times, most recently from f85c88f to f32c205 Compare March 12, 2019 12:31
@njgheorghita njgheorghita force-pushed the refactor-jsonrpc-endpoints branch from f32c205 to 889c1cf Compare March 12, 2019 12:48
@njgheorghita njgheorghita merged commit e4f46cf into ethereum:master Mar 12, 2019
@njgheorghita njgheorghita deleted the refactor-jsonrpc-endpoints branch March 12, 2019 15:39
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