-
Notifications
You must be signed in to change notification settings - Fork 303
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 support for Polygon to use Blocknative #748
Add support for Polygon to use Blocknative #748
Conversation
Network.RINKEBY, | ||
Network.ROPSTEN, | ||
Network.GÖRLI, | ||
Network.POLYGON |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've also added the other Ethereum testnets along with Polygon
@evgenyboxer did we add any Polygon specific logic to compensate for not having blocknative that we should now remove? |
@garethfuller I've scanned the code and it doesn't seem we do anything special for Polygon (except hiding/disabling some features). Blocknative is mostly used for balance refetching and since everything is going through |
… into evgeny/ui-724-update-bnc-sdk-to-support-polygon
Description
The latest release of bnc-sdk adds support for Polygon 🎉
Type of change
How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.
Checklist:
master
if hotfix,develop
if not