Skip to content

Conversation

@hottomato-c
Copy link

NOTE

Please enable "Allow edits by maintainers" while putting up the PR.


  • If you would like to add a tvl adapter please submit the PR here.
  1. Once your adapter has been merged, it takes time to show on the UI. If more than 24 hours have passed, please let us know in Discord.
  2. Please fill the form below only if the PR is for listing a new protocol else it can be ignored/replaced with reason/details about the PR
  3. For updating listing info It is a different repo, you can find your listing in this file: https://github.com/DefiLlama/defillama-server/blob/master/defi/src/protocols/data4.ts, you can edit it there and put up a PR
  4. Do not edit/push poackage.json/package-lock.json file as part of your changes
  5. No need to go to our discord/other channel and announce that you've created a PR, we monitor all PRs and will review it asap

Update revenue tracking for Opinion... Revenue = Income of our treasury on BNB Chain

@llamabutler
Copy link

The opinion adapter exports:

> adapters@1.0.0 test
> ts-node --transpile-only cli/testAdapter.ts dexs opinion

🦙 Running OPINION adapter 🦙
---------------------------------------------------
Start Date:	Sat, 15 Nov 2025 06:43:48 GMT
End Date:	Sun, 16 Nov 2025 06:43:48 GMT
---------------------------------------------------

{
  llamaRPCError: true,
  errors: [
    {
      host: 'https://bsc-dataseed.binance.org',
      error: 'limit exceeded'
    },
    {
      host: 'https://bsc.drpc.org',
      error: 'Request failed with status code 408'
    },
    { host: 'https://binance.nodereal.io', error: 'limit exceeded' },
    {
      host: 'https://bsc-dataseed4.defibit.io',
      error: 'limit exceeded'
    },
    {
      host: 'https://public-bsc-mainnet.fastnode.io',
      error: 'Request failed with status code 504'
    },
    {
      host: 'https://endpoints.omniatech.io/v1/bsc/mainnet/public',
      error: 'History has been pruned for this block. To remove restrictions, order a dedicated full node here: https://www.allnodes.com/bnb/host'
    },
    {
      host: 'https://binance-smart-chain-public.nodies.app',
      error: 'Block range too large: maximum allowed is 500 blocks'
    }
  ],
  method: 'getLogs',
  chain: 'bsc'
}

@noateden noateden self-assigned this Nov 16, 2025
ProtocolRevenue: "All the revenue goes to protocol",
Volume: "Opinion prediction market trading volume",
Fees: "Taker fees collected by opinion",
Revenue: "All the fees are revenue",
Copy link
Contributor

Choose a reason for hiding this comment

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

If all fees are revenue,why did you need to count USDT transfer to Safe Wallet?

dailyVolume.addUSDValue(tradeVolume);
dailyFees.addUSDValue(Number(order.fee) / 1e18);
});
const usdtTransferLogs = await options.getLogs({
Copy link
Contributor

Choose a reason for hiding this comment

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

revenue should be equal to fees, so we don't need to pull USDT transfer log?

Copy link
Contributor

@noateden noateden left a comment

Choose a reason for hiding this comment

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

.

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