You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
<!-- This is an auto-generated comment: summarize by coderabbit.ai --> #1
A new Ethereum TVL adapter for the MONY token is introduced. It queries the totalSupply of the MONY ERC-20 contract, scales the result to match USDC decimal precision, and reports the total value to the adapter API.
Changes
Cohort / File(s)
Summary
New MONY TVL Adapter projects/jpmorgan-mony/index.js
New adapter file that defines MONY token constants, loads base asset addresses from core assets helper, and implements an async ethereum.tvl function to query token supply, scale decimals, and report results to USDC.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~8 minutes
Poem
🐰 A shiny new adapter hops into the fold,
Tracking MONY's supply, worth more than gold!
Scale those decimals, report the gains,
TVL flows through the blockchain veins! ✨
🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
Check name
Status
Explanation
Resolution
Description check
⚠️ Warning
The description provides context about what MONY is and what the adapter tracks, but is missing most of the required template sections for new listings.
Fill in the missing required template sections including Name, Twitter Link, Website Link, Chain, Current TVL, methodology, Category, and other applicable fields from the template.
✅ Passed checks (4 passed)
Check name
Status
Explanation
Title check
✅ Passed
The title clearly and specifically describes the main change: adding a JPMorgan MONY adapter for a tokenized money market fund, which matches the file addition.
Docstring Coverage
✅ Passed
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check
✅ Passed
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
✅ Passed
Check skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches🧪 Generate unit tests (beta)
Create PR with unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Activity