-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix: add missing amms for jupiter tracking #7131
base: main
Are you sure you want to change the base?
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
@Hosuke lmk if there are any fixes needed here. Would be great to get this merged |
Hey just wanted to follow up here - really excited to get this merged and happy to do whatever necessary to accelerate. |
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 @rajivpo
(Sorry for the late reply, was in devcon last few days)
Description:
There are DEXes being routed to through Jupiter which have significant volume (SolFi, Obric V2) that are not currently being tracked. The newly added DEX labels/program ids comes from
https://station.jup.ag/api-v6/get-program-id-to-label
as per existing comments.quick links for more information: