Skip to content

Conversation

@VaitaR
Copy link
Owner

@VaitaR VaitaR commented Oct 9, 2025

🎯 Changes Summary

This PR removes deprecated OKLink/XLayer scanner implementations and updates documentation to use named parameters for better clarity.

βœ… What was done:

  1. Removed OKLink/XLayer references - No longer supported scanner implementations
  2. Removed deprecated BaseScanV1 - Base network now supported via Etherscan V2
  3. Updated documentation - All examples now use named parameters for clarity
  4. Updated docstrings - Reflect current scanner support
  5. Removed OKLink parser - Cleaned up unused code
  6. Updated test configurations - Reflect current scanner capabilities

πŸ”§ Documentation Improvements:

Before (unclear):

client = ChainscanClient.from_config('blockscout', 'v1', 'blockscout_eth', 'eth')

After (clear):

client = ChainscanClient.from_config(
    scanner_name='blockscout',      # Provider name
    scanner_version='v1',           # API version
    scanner_id='blockscout_eth',    # Config identifier for Ethereum
    network='eth'                   # Network name
)

πŸ“‹ Current Scanner Support:

Provider scanner_name scanner_version scanner_id network API Key
BlockScout ❌ Not required
Etherscan βœ…
Etherscan Base βœ…
Moralis βœ…

πŸŽ‰ Benefits:

  • 🧹 Cleaner codebase - Removed deprecated scanner implementations
  • πŸ“š Better documentation - Named parameters make usage clear
  • πŸ”§ Improved DX - Less confusion about parameter order
  • πŸš€ Future-ready - Base network properly integrated via Etherscan V2

βœ… Testing:

  • βœ… All existing tests pass
  • βœ… Documentation examples work correctly
  • βœ… Base network accessible via Etherscan V2
  • βœ… No breaking changes for existing functionality

…d parameters

- Remove all references to OKLink and XLayer scanner implementations
- Remove deprecated BaseScanV1 scanner (Base now supported via Etherscan V2)
- Update documentation to use named parameters in examples
- Update docstrings to reflect current scanner support
- Remove OKLink parser and related code
- Update test configurations to reflect current scanner support
- Update README and AGENTS.md with current scanner capabilities
@VaitaR VaitaR merged commit 66ec1a5 into main Oct 9, 2025
12 checks passed
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