Skip to content

Fix contract getters and payout regressions - #748

Merged
chonilius merged 1 commit into
MergeFi:mainfrom
cristianooR7:fix/issues-447-450-451-452
Oct 20, 2026
Merged

chonilius merged 1 commit into
MergeFi:mainfrom
cristianooR7:fix/issues-447-450-451-452

Conversation

@cristianooR7

Copy link
Copy Markdown

Summary

Expose fixed contract configuration through on-chain getters, bound payout recipients, centralize instance TTL extension, and remove duplicate README history sections.

Related issues

Closes #450
Closes #451
Closes #447
Closes #452

Changes

  • Add grace-period and basis-point denominator getters to escrow and milestones, and denominator, max-deposit, and inactivity-window getters to maintenance-pool.
  • Add shared MAX_RECIPIENTS enforcement with typed errors and boundary coverage.
  • Move instance-storage TTL extension into mergefi-common with named thresholds and ledger-time derivation documentation.
  • Keep a single README “Recent Changes” section and direct history to CHANGELOG.md.

Test plan

  • Existing tests pass (make test)
  • New tests added for changed behaviour
  • Manually verified against a local node / testnet where applicable

Tests not run, per request.

Checklist

  • Branch is up to date with main
  • No debug / dead code left in
  • cargo fmt and cargo clippy are clean

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@drips-wave

drips-wave Bot commented Oct 19, 2026

Copy link
Copy Markdown

@cristianooR7 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@chonilius
chonilius merged commit f631642 into MergeFi:main Oct 20, 2026
0 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants