Skip to content

Feature/escrow investor allowlist#134

Merged
mikewheeleer merged 3 commits intoLiquifact:mainfrom
michaelvic123:feature/escrow-investor-allowlist
Apr 1, 2026
Merged

Feature/escrow investor allowlist#134
mikewheeleer merged 3 commits intoLiquifact:mainfrom
michaelvic123:feature/escrow-investor-allowlist

Conversation

@michaelvic123
Copy link
Copy Markdown
Contributor

closes #96

- Add DataKey::AllowlistEnabled and DataKey::InvestorAllowed(Address)
- Add enable_allowlist / disable_allowlist (admin-only, emits AllowlistChanged event)
- Add add_to_allowlist / remove_from_allowlist (admin-only, O(1) per-address storage)
- Add is_allowlisted / is_allowlist_enabled read helpers
- Gate fund_impl: when AllowlistEnabled is true, reject callers not in the list
- Gate applies to both fund() and fund_with_commitment()
- Default is disabled  existing open-round escrows are unaffected
- Entries persist across disable/re-enable cycles
- Add escrow/src/test_allowlist.rs with 18 tests covering:
  defaults, enable/disable, add/remove, auth guards, fund gating,
  fund_with_commitment gating, re-enable persistence, multi-investor isolation
- Update README with allowlist API table and gas note
@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 28, 2026

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

@mikewheeleer mikewheeleer merged commit 870c3e5 into Liquifact:main Apr 1, 2026
1 check failed
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 Investor Whitelist Gate Before Funding

2 participants