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

feat: switch to V2 API and implement supported_chain for broader network support #156

Merged
merged 7 commits into from
Nov 26, 2024

Conversation

antazoey
Copy link
Member

@antazoey antazoey commented Oct 17, 2024

What I did

Implement the API defined here ApeWorX/ape#2325
and use their provided API for getting the list

How I did it

use the API they document to get the list

How to verify it

ape console --network https://arbitrum-nova.publicnode.com

then

In [1]: provider.network.explorer
Out[1]: Etherscan(name='etherscan', network=<ethereum:custom chain_id=42170>)

Checklist

  • Passes all linting checks (pre-commit and CI jobs)
  • New test cases have been added and are passing
  • Documentation has been updated
  • PR title follows Conventional Commit standard (will be automatically included in the changelog)

Copy link

This pull request is considered stale because it has been open 30 days with no activity. Remove stale label, add a comment, or make a new commit, otherwise this PR will be closed in 5 days.

@github-actions github-actions bot added the stale label Nov 17, 2024
@antazoey
Copy link
Member Author

required another release of ape for ApeWorX/ape#2380

@github-actions github-actions bot removed the stale label Nov 21, 2024
@antazoey antazoey changed the title feat: implemenet supports_chain() API for adhoc network support feat: switch to V2 API and implement supported_chain for broader network support Nov 21, 2024
ape_etherscan/utils.py Outdated Show resolved Hide resolved
@antazoey antazoey merged commit 6bff6f9 into ApeWorX:main Nov 26, 2024
6 of 14 checks passed
@antazoey antazoey deleted the feat/auto-adhoc branch November 26, 2024 19:23
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