-
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
add justbet fees & volume aggregator #2706
Conversation
The justbet adapter exports:
|
The justbet adapter exports:
|
The justbet adapter exports:
|
The justbet adapter exports:
|
I added winr rpc to .env file while running tests |
fees/justbet/index.ts
Outdated
|
||
return { | ||
revenue: amountUsd, | ||
fees: (amountUsd * 60) / 100, |
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.
hi, how can the revenue be higher than the fees charged to the user?
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.
you’re right, the fee calculation should be revenue, and the revenue calculation should be a fee. I will change it.
The justbet adapter exports:
|
aggregators/justbet/index.ts
Outdated
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.
this is not an aggregator, closer to a dex?
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.
also, any idea why I am getting negative value for today after the refactor?
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.
Yes, you’re right. I’ll start by moving to the DEX section first. The fromApi caches cause the returnEpochResultInUsdByAddress function to always calculate using the latest price of the vault token address. As a result, volume fluctuations appear negative due to price changes. To address this, we should use the returnEpochResultInActiveEpochByAddress function and convert the values to USD using the same price for both yesterday and today. I'll send updates
The justbet adapter exports:
|
fix: re calculate fees
The justbet adapter exports:
|
The justbet adapter exports:
|
The justbet adapter exports:
|
The justbet adapter exports:
|
No description provided.