-
Notifications
You must be signed in to change notification settings - Fork 152
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 market name #1221
Fix market name #1221
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Looks good. Just left one comment about a place where we might still need to import the getMarketName
function. Was also wondering if we could maybe make the getMarketName
function only take in a FuturesMarketAsset
, to make sure that we don't misuse it in the future. I understand that that might lead to a number of type fixes that might be beyond the scope of this PR though.
Add the
[ASSET]-PERP
market name to the market object, and replace all instances to use the market info or util function.Description
Replace all market name instances with a version created by utility function.
Related issue