Skip to content

feat(fee): emit versioned events for init, set_fee, deposit, and withdraw (#1189) - #1377

Closed
lienhoatienphung-svg wants to merge 1 commit into
CalloraOrg:mainfrom
lienhoatienphung-svg:feature/1189-fee-lifecycle-events
Closed

lienhoatienphung-svg wants to merge 1 commit into
CalloraOrg:mainfrom
lienhoatienphung-svg:feature/1189-fee-lifecycle-events

Conversation

@lienhoatienphung-svg

Copy link
Copy Markdown

Summary

Emits structured, versioned events following the (name, v1, subject) topic convention across all state-changing entrypoints in FeeContract (init, set_fee, deposit, and withdraw).

Key Changes

  • contracts/fee/src/lib.rs:
    • init: emits ("fee_init", "v1", admin) with initial 0u32 fee bps.
    • set_fee: emits ("fee_set", "v1", admin) with (old_fee_bps, fee_bps).
    • deposit: emits ("fee_deposited", "v1", caller) with (amount, new_accumulated).
    • withdraw: emits ("fee_withdrawn", "v1", admin) with (recipient, amount).
    • Adds test_events_emitted_for_init_set_fee_deposit_and_withdraw and test_events_not_emitted_on_zero_or_negative_deposit.

Closes #1189


Payout Wallet (Base USDC / EVM): 0x3806cd592a508d1992Ed007C2F53de251C2E7452

@lienhoatienphung-svg
lienhoatienphung-svg force-pushed the feature/1189-fee-lifecycle-events branch from 8a0c0fb to fff1b50 Compare October 16, 2026 07:18
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.

Announce fee configuration and deposit changes

2 participants