-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Zapzy Fix #4216
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
base: master
Are you sure you want to change the base?
Zapzy Fix #4216
Conversation
The zapzy adapter exports:
|
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.
I dont understand the change, why exclude referral fees? it is not going to the protocol, so should not be revenue?
The referral feature has been removed and replaced by a 500% increase of the creator fee. The all time referral earnings were less than 0.1 SOL which is the reason I fully excluded it. |
The zapzy adapter exports:
|
hmm, I have refactored, looks like the frontend now returns daily data instead of accumulative data each data? Maybe it is better if the adapter is changed to use dune query, this way, it is easier for us to verify/refill 🙏 |
The zapzy adapter exports:
|
done, now using a dune query instead of our api |
Changed the Revenue Calculation and removed referral.