Skip to content

feat: gate revenue_deposit_contract.deposit on offering allowlist#508

Open
Danielodingz wants to merge 1 commit into
RevoraOrg:masterfrom
Danielodingz:feat/deposit-contract-allowlist
Open

feat: gate revenue_deposit_contract.deposit on offering allowlist#508
Danielodingz wants to merge 1 commit into
RevoraOrg:masterfrom
Danielodingz:feat/deposit-contract-allowlist

Conversation

@Danielodingz

Copy link
Copy Markdown
Contributor

Close #481

Summary

  • add an admin-managed allowlist for authorized offering contract addresses
  • gate revenue deposits so only approved callers can fund periods
  • reject unauthorized callers with UnauthorizedDepositor
  • emit dep_allow_add and dep_allow_rm events for allowlist changes
  • add regression tests covering unauthorized, authorized, and empty-allowlist cases
  • document the new deposit guard in the revenue deposit contract docs

Testing

  • attempted targeted Rust test execution for the new regression cases
  • local verification is currently blocked by existing repository-wide compile issues in the main contract module, so full cargo test validation could not be completed in this environment

@drips-wave

drips-wave Bot commented Jun 29, 2026

Copy link
Copy Markdown

@Danielodingz 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

@Danielodingz

Copy link
Copy Markdown
Contributor Author

Hello @thlpkee20-wq please review PR and merge

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.

Add cross-contract guard for revenue_deposit_contract callable-only-by-offering allowlist

1 participant