Skip to content

feat(reward-pool): add approved spender revocation - #105

Merged
Kaylahray merged 1 commit into
learnault:mainfrom
chijioke003:feat/revoke-approved-spender
Jul 29, 2026
Merged

feat(reward-pool): add approved spender revocation#105
Kaylahray merged 1 commit into
learnault:mainfrom
chijioke003:feat/revoke-approved-spender

Conversation

@chijioke003

@chijioke003 chijioke003 commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds support for revoking approved RewardPool spenders.

Changes

  • Add remove_approved_spender to the RewardPool interface.
  • Restrict spender revocation to the stored, authenticated admin.
  • Mark revoked spender entries as false in persistent storage.
  • Emit a SpenderRemoved event after successful revocation.
  • Add coverage confirming:
    • revoked spenders cannot distribute rewards;
    • revoking one spender does not affect other approved spenders;
    • non-admin revocation attempts fail and preserve approval;
    • repeated revocation succeeds and leaves the spender unauthorized.
  • Add generated Soroban test snapshots for the new cases.

Testing

cargo test -p reward-pool

Closes #92

@Kaylahray
Kaylahray merged commit 1a397cd into learnault:main Jul 29, 2026
1 check 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

Development

Successfully merging this pull request may close these issues.

[RewardPool] Revoke approved spender access

2 participants