fix(cooler): refresh loans docs for current contracts - #229
Conversation
|
Someone is attempting to deploy a commit to the Olympus Association Team on Vercel. A member of the Team first needs to authorize it. |
|
@coderabbitai review |
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThis PR updates the Cooler Loans documentation to align with Cooler V2 mechanics and governance. Changes clarify oracle-backed LTVs, replace V1-specific language, refine loan and repayment examples, add legacy migration guidance, and correct typos in the FAQ. ChangesCooler Loans V2 Documentation Alignment
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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. Comment |
✅ Actions performedReview triggered.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/overview/04_cooler-loans.md`:
- Line 155: The sentence "Interest is charged on an accrual basis. To get your
principal back you will need to pay interest in full." is confusing—replace it
with wording that clarifies borrowers repay the debt (principal) and that
collateral becomes releasable once accrued interest and any outstanding amounts
are paid; update the sentence in the docs (the line containing "To get your
principal back you will need to pay interest in full.") to something like:
"Interest accrues on the loan; you must pay the accrued interest (and any
outstanding principal) in full to satisfy the debt and have your collateral
released." Ensure the new sentence replaces the original exact line.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: c6da28a4-0e71-46de-8793-2929a2622392
📒 Files selected for processing (1)
docs/overview/04_cooler-loans.md
6ca647e to
f85023f
Compare
f85023f to
0977da7
Compare
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
1 participant
Summary
Verification
pnpm exec prettier --check docs/overview/04_cooler-loans.mdgit diff --checkpnpm buildsucceeds; repo has pre-existing broken-link warnings unrelated to this pageNotes
Current Cooler V2 contract addresses were checked against the existing docs/contract references and Olympus Cooler contract registry notes. The Migrator is intentionally kept only in the legacy contracts table so the docs do not present it as part of the normal/current borrowing surface.
Summary by CodeRabbit