-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
adding m-btc and solvBTC prices in prices_linea_tokens.sql #6237
Conversation
adding m-btc and solvBTC on linea
Workflow run id 9623628911 approved. |
Workflow run id 9623629181 approved. |
Workflow run id 9623629198 approved. |
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.
LGTM.✅
Thank you @0xroll
@@ -26,7 +26,8 @@ def __init__(self): | |||
"zksync": "zksync-zksync", | |||
"zora": "eth-ethereum", | |||
"mantle": "mnt-mantle", | |||
"blast": "blast-blast" | |||
"blast": "blast-blast", | |||
"linea": "eth-ethereum" |
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.
Adding linea support similar to zora.
(Currently linea and zora are still not in the coinpaprika list: https://api.coinpaprika.com/v1/contracts/)
@@ -24,4 +24,6 @@ FROM | |||
,('uni-uniswap', 'linea', 'UNI', 0x636B22bC471c955A8DB60f28D4795066a8201fa3, 18) | |||
,('link-chainlink', 'linea', 'LINK', 0x5B16228B94b68C7cE33AF2ACc5663eBdE4dCFA2d, 18) | |||
,('dai-dai', 'linea', 'DAI', 0x4AF15ec2A0BD43Db75dd04E62FAA3B8EF36b00d5, 18) | |||
,('m-btc-merlins-seal-btc', 'linea', 'M-BTC',0xe4D584ae9b753e549cAE66200A6475d2f00705f7, 18) |
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.
@@ -24,4 +24,6 @@ FROM | |||
,('uni-uniswap', 'linea', 'UNI', 0x636B22bC471c955A8DB60f28D4795066a8201fa3, 18) | |||
,('link-chainlink', 'linea', 'LINK', 0x5B16228B94b68C7cE33AF2ACc5663eBdE4dCFA2d, 18) | |||
,('dai-dai', 'linea', 'DAI', 0x4AF15ec2A0BD43Db75dd04E62FAA3B8EF36b00d5, 18) | |||
,('m-btc-merlins-seal-btc', 'linea', 'M-BTC',0xe4D584ae9b753e549cAE66200A6475d2f00705f7, 18) | |||
,('solvbtc-solv-protocol-solvbtc', 'linea', 'solvBTC', 0x5FFcE65A40f6d3de5332766ffF6A28BF491C868c, 18) |
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.
Congrats, your important contribution to this open-source project has earned you a GitPOAP! GitPOAP: 2024 Dune Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
adding m-btc and solvBTC on linea
Thank you for contributing to Spellbook 🪄
Contribution type
Please check the type of contribution this pull request is for:
For adding to existing spell lineage
If you are adding to an existing spell lineage, please provide the following information:
Additional information
Please provide any additional information that might help us review your pull request:
https://api.coinpaprika.com/v1/coins/m-btc-merlins-seal-btc
https://api.coinpaprika.com/v1/coins/solvbtc-solv-protocol-solvbtc