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.

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

Update routerAddresses.ts: add solana drift protocol #733

Merged
merged 3 commits into from
Sep 22, 2023

Conversation

0xbigz
Copy link
Contributor

@0xbigz 0xbigz commented Aug 11, 2023

add solana's drift protocol program ids to routerAddresses.ts

(a follow up on DefiLlama/DefiLlama-Adapters#6356)
todo: what should be in "id" field?

add solana's drift protocol
@0xbigz 0xbigz changed the title Update routerAddresses.ts Update routerAddresses.ts: add solana drift protocol Aug 11, 2023
@dtmkeng
Copy link
Contributor

dtmkeng commented Aug 15, 2023

@0xngmi

@0xbigz
Copy link
Contributor Author

0xbigz commented Sep 22, 2023

@dtmkeng bumping here :)

i pull/resolved conflict with master

@@ -2676,5 +2676,15 @@ export default ([
]
}
},
{
"id":"3388", // todo?
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is the id set to 3388? drift's id is 970

the protocol with id 3388 is sanctum

@0xngmi 0xngmi merged commit ae993c1 into DefiLlama:master Sep 22, 2023
2 checks passed
@0xngmi
Copy link
Member

0xngmi commented Sep 22, 2023

merged, however we dont have infra support for solana yet so data wont appear right away

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants