-
Notifications
You must be signed in to change notification settings - Fork 5
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: migrate to Unified Bridge, introduce the ICTT bridge #82
base: main
Are you sure you want to change the base?
Conversation
refactor: use @avalabs/bridge-unified for all bridging
c6c3a08
to
10f45d4
Compare
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.
Unrelated to this changes, just FYI: the tokenlist
endpoint responds with an empty array fro sepolia testnet, so it's only possible to use ETH for briding from sepolia to fuji C.
My sepolia -> fuji C ethereum bridge finished successfully, however the tracker just keeps trying 😅
And the BTC bridge transaction is not recognized in the activity log
Do we want to address these issues in this PR?
Description
@avalabs/bridge-sdk
(in the frontend -- some backend parts need to stay for a while)@avalabs/bridge-unified
(comes in with CTTP, ICTT (erc20-erc20) and Avalanche Bridge integrations)Changes
@avalabs/bridge-sdk
are still left - during the migration phase, some people may have oldBridgeTransaction
s either in progress or completed, but not acknowledged in the UI yet. We may need a schema migration, depending on how the UnifiedBridgeSDK handles BTC tracking.useAvalancheBridge()
/useBtcBridge()
/useEthBridge()
/useUnifiedBridge()
hooks. The olduseBridge()
which previously merged the capabilities of them all is now enough to handle everything.BridgeFormETH
/BridgeFormBTC
/BridgeFormAVAX
orBridgeFormUnified
. TheBridgeForm
got promoted and is used directly byBridge.tsx
page.The new user path
Testing
unified-bridge-cctp
unified-bridge-ictt
unified-bridge-ab-evm
unified-bridge-ab-ava-to-btc
unified-bridge-ab-btc-to-ava
Screenshots:
Feature Flags examples
When Bitcoin -> Avalanche bridge IS disabled
The moment when selected bridge route (Avalanche -> Bitcoin) gets disabled
Screen.Recording.2024-11-06.at.12.23.58.mov
Checklist for the author