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.

Sign up for GitHub

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

Jump to bottom

Hyperliquid route #3239

Open
wants to merge 34 commits into
base: development
Choose a base branch
from
Open

Hyperliquid route #3239

wants to merge 34 commits into from
+1,592 −1,249

Conversation

emreboga
Copy link
Collaborator

@emreboga emreboga commented Jan 29, 2025 •
edited
Loading

This PR adds Hyperliquid route, Hyperliquid chain and enables the Hyperliquid deposit UX flow. Please see the loom below for details.

PR walkthrough:
https://www.loom.com/share/47b64c7586c84fc5b4914d972f260e88?sid=46f72b22-0ab9-484f-9d89-fb8149de756a

UX demo:
https://www.loom.com/share/14b3c1b7ba4547d0be0dbc84b4b973ad?sid=c470625d-3be4-4e1c-82e1-dbe7111487c9

@netlify Netlify
Copy link

netlify bot commented Jan 29, 2025 •
edited
Loading

✅ Deploy Preview for wormhole-connect ready!

Name Link
🔨 Latest commit a5393b2
🔍 Latest deploy log https://app.netlify.com/sites/wormhole-connect/deploys/67acb635eeb17400083b28a1
😎 Deploy Preview https://deploy-preview-3239--wormhole-connect.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify Netlify
Copy link

netlify bot commented Jan 29, 2025 •
edited
Loading

✅ Deploy Preview for wormhole-connect-mainnet ready!

Name Link
🔨 Latest commit a5393b2
🔍 Latest deploy log https://app.netlify.com/sites/wormhole-connect-mainnet/deploys/67acb635e1a18200087b26d4
😎 Deploy Preview https://deploy-preview-3239--wormhole-connect-mainnet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@emreboga emreboga marked this pull request as ready for review January 29, 2025 16:45
@emreboga emreboga requested review from artursapek and kev1n-peters and removed request for artursapek January 31, 2025 20:16
artursapek
artursapek reviewed Feb 5, 2025
View reviewed changes
wormhole-connect/src/hooks/useComputeDestinationTokens.ts Outdated Show resolved Hide resolved
wormhole-connect/src/hooks/useFetchSupportedRoutes.ts Outdated Show resolved Hide resolved
wormhole-connect/src/routes/hyperliquid/index.ts Outdated Show resolved Hide resolved
wormhole-connect/src/routes/hyperliquid/index.ts Outdated Show resolved Hide resolved
wormhole-connect/src/routes/hyperliquid/index.ts Show resolved Hide resolved
wormhole-connect/src/routes/hyperliquid/index.ts Outdated Show resolved Hide resolved
wormhole-connect/src/utils/errors.ts Outdated Show resolved Hide resolved
wormhole-connect/src/views/v2/Bridge/AssetPicker/ChainList.tsx Outdated Show resolved Hide resolved
@emreboga emreboga requested a review from artursapek February 6, 2025 12:39
@emreboga emreboga requested a review from kev1n-peters February 6, 2025 17:03
@emreboga emreboga force-pushed the emre/HL-route branch 4 times, most recently from ce96211 to b7bfc67 Compare February 11, 2025 17:09
@emreboga
Hyperliquid route
0ca4a11 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Add UI hacks to add HYPE chain
fc40204 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Add a new config for non-SDK chains
2230722 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Add Arb check for source chain in the route
c81dad2 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Show HL route only when HL chain is selected
b592c43 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Remove Arb check for source chain in the route
001ae15 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Don't show HL when Arb selected as source
8892499 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Temp fix for Receipt types until Mayan upgrades Wormhole SDK version
affb099 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
HL specific messaging on route card
66aada8 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Show/hide HL route when HL chain is selected as destination
1cfd25e 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Show only USDC for Hyperliquid as destination
51e1462 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Remove providedBy text
8858291 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
HL specific messaging on Redeem view
1b21c56 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
HL specific messaging on Widget
1e6933e 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Render explorer link according to Hyperliquid tx status
3d2f5a8 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
HL specific error messaging for manual claim failure
1d8cfb3 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
HL specific error messaging on Transaction details card
20b25e1 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Fix destination chain icon in redeem view
3e61d03 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Filter supported routes for HL and fix Route card styling
01d8a0a 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Refactor
ee45301 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Fix the HL quote to error when output amount is below 5
d130f97 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Addressing the first round of review feedback
23a6b73 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Use Network type instead
ee80cd1 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Disable swap-inputs for HL
5ed425d 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Replace hardcoded USDC contract on Arb
2c47ffb 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Use latest quote to determine received amount
9b656ea 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Use toAmount in the final receipt instead of the one in final quote
74bbb3f 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Add default gas drop to Arbitrum
32f7aa8 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Fix build errors after a rebase
c43ec55 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Update route card messaging for HL
97639e4 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@emreboga
Remove gas message and fix HL explorer link
971e5dd 
Signed-off-by: Emre Bogazliyanlioglu <[email protected]>
@socket-security Socket Security
Copy link

New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@aptos-labs/[email protected] 🔁 npm/@aptos-labs/[email protected] None +9 22.1 MB gregnazario
npm/@aptos-labs/[email protected] 🔁 npm/@aptos-labs/[email protected] Transitive: environment, network +39 8.3 MB aptos-labs
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None +92 10.3 MB existentialism, hzoo, jlhwung, ...1 more
npm/@mayanfinance/[email protected] network +12 18.9 MB mohandes
npm/@metaplex-foundation/[email protected] 🔁 npm/@metaplex-foundation/[email protected] None +1 1.98 MB svc01-metaplex
npm/@types/[email protected] 🔁 npm/@types/[email protected] None 0 2.22 MB types
npm/@vitejs/[email protected] 🔁 npm/@vitejs/[email protected] Transitive: environment, shell +14 426 MB vitebot

View full report↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@artursapek artursapek Awaiting requested review from artursapek

@kev1n-peters kev1n-peters Awaiting requested review from kev1n-peters

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@emreboga @artursapek @kev1n-peters