-
Notifications
You must be signed in to change notification settings - Fork 408
kaspabridge added #438
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
base: master
Are you sure you want to change the base?
kaspabridge added #438
Conversation
| }, | ||
|
|
||
| { | ||
| id: 102, |
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.
starkgate should keep its id - 102
create new object for kaspabridge with id 103
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.
updated!
src/data/bridgeNetworkData.ts
Outdated
| displayName: "Kaspa Bridge", | ||
| bridgeDbName: "kaspabridge", | ||
| slug: "kaspabridge", | ||
| iconLink: "https://app.kaspafinance.io/images/chains/0xf87e587ab945f7b111329a6ace6dc497d34f098b.png", |
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.
pls create pr with ypur icon here
https://github.com/DefiLlama/icons
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.
PR created
src/adapters/kaspabridge/index.ts
Outdated
| throw new Error(`Missing kaspabridge config for chain ${chain}`); | ||
| } | ||
|
|
||
| const provider = new providers.JsonRpcProvider(rpcUrl); |
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.
need to use defillama sdk (public sdk doesnt have old blocks)
reason: 'processing response error',
code: 'SERVER_ERROR',
body: '{"jsonrpc":"2.0","id":44,"error":{"code":-32701,"message":"History has been pruned for this block. To remove restrictions, order a dedicated full node here: https://www.allnodes.com/bnb/host"}}\n',
since this bridge is 2 way u can do similar to arbitrum bridge
set kasplex as destination chain and track events only on bsc
src/adapters/kaspabridge/index.ts
Outdated
| throw new Error(`Missing kaspabridge config for chain bsc`); | ||
| } | ||
|
|
||
| const provider = new providers.JsonRpcProvider(rpcUrl); |
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.
use defillama sdk please
check other adapters for references
src/adapters/kaspabridge/index.ts
Outdated
| blockNumber: log.blockNumber, | ||
| from: "0x0000000000000000000000000000000000000000", | ||
| to: "0x0000000000000000000000000000000000000000", | ||
| token: "0x0000000000000000000000000000000000000000", |
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.
why token, from and to hardcoded to zero address?
The Kaspa Bridge has been added to the bridges-server to handle transactions on the Kaspa blockchain. This bridge supports bidirectional token transfers between Binance Smart Chain (BSC) and Kasplex: BSC to Kasplex and Kasplex to BSC. please Merge this PR.
Here is the my kaspaBridge site Link: https://www.kaspabridge.com
chains: kasplex , bsc
Logo (High resolution): https://app.kaspafinance.io/images/chains/0xf87e587ab945f7b111329a6ace6dc497d34f098b.png