Skip to content

Conversation

@zeevick10
Copy link

Summary

Replace outdated GitHub wiki reference with official ethereum.org documentation link for eth_sign JSON-RPC method.

Changes

  • Before: https://github.com/ethereum/wiki/wiki/JSON-RPC#eth_sign
  • After: https://ethereum.org/en/developers/docs/apis/json-rpc/#eth_sign

Why this change?

  • Official source: ethereum.org is the current official documentation hub
  • Better maintenance: Links to ethereum.org are more stable and maintained

File affected

  • internal/ethapi/api.go (line 1552)

Testing

  • Link accessibility verified
  • Documentation content matches previous reference
  • No functional code changes

@rjl493456442
Copy link
Member

Could you please also fixes the other outdated links? Like searching with github.com/ethereum/wiki/wiki gives you 4 links

@zeevick10
Copy link
Author

Could you please also fixes the other outdated links? Like searching with github.com/ethereum/wiki/wiki gives you 4 links

@rjl493456442 Done!

I've updated all 3 outdated github.com/ethereum/wiki/wiki links found in the codebase
Thanks for the feedback! 🙏

@lightclient lightclient changed the title docs: Update JSON-RPC documentation link to official ethereum.org all: fix outdated ethereum wiki json-rpc json-rpc doc links Jul 14, 2025
@lightclient lightclient added this to the 1.16.2 milestone Jul 14, 2025
@lightclient lightclient merged commit 5bce990 into ethereum:master Jul 14, 2025
4 of 5 checks passed
howjmay pushed a commit to iotaledger/go-ethereum that referenced this pull request Aug 27, 2025
…#32209)

Replace outdated wiki reference with ethereum.org
documentation links
maoueh pushed a commit to streamingfast/go-ethereum that referenced this pull request Sep 4, 2025
…#32209)

Replace outdated wiki reference with ethereum.org
documentation links
gballet pushed a commit to gballet/go-ethereum that referenced this pull request Sep 11, 2025
…#32209)

Replace outdated wiki reference with ethereum.org
documentation links
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.

3 participants