Skip to content

Expose FlowFees fee receiver addresses via getter and change event - #615

Merged
janezpodhostnik merged 3 commits into
masterfrom
janez/fee-receiver-addresses
Jul 24, 2026
Merged

Expose FlowFees fee receiver addresses via getter and change event#615
janezpodhostnik merged 3 commits into
masterfrom
janez/fee-receiver-addresses

Conversation

@janezpodhostnik

@janezpodhostnik janezpodhostnik commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Closes #614

  • Add FlowFees.getFeeReceiverAddresses(): returns the addresses that may receive
    transaction fee deposits (the FlowFees account plus all child fee accounts), so
    integrators can fetch the receiver set with one stable script instead of
    entitlement-sensitive storage reads or hardcoded per-network lists.
  • Add a ChildFeeAccountsChanged(addresses: [Address]) event, emitted with the full
    new list whenever child fee accounts are added.
  • Route the admin mutation through the new
    Administrator.createChildFeeAccounts(count:payer:) instead of a raw storage write
    in the transaction. The contract creates the accounts itself, so the stored
    capabilities are valid, keyless, and fully owned by FlowFees by construction.

Add FlowFees.getFeeReceiverAddresses() returning the FlowFees account
plus all child fee accounts, and emit ChildFeeAccountsChanged whenever
the child fee account list changes.

Child fee accounts are now created inside the new
Administrator.createChildFeeAccounts(count:payer:) function instead of
being assembled in the transaction and written to storage directly,
which guarantees they are keyless, fully owned by FlowFees, and valid
fee deposit targets by construction.

Closes #614
@janezpodhostnik janezpodhostnik self-assigned this Jul 24, 2026
@janezpodhostnik
janezpodhostnik requested review from a team, SupunS and turbolent July 24, 2026 15:04
Comment thread contracts/FlowFees.cdc Outdated

@turbolent turbolent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@janezpodhostnik
janezpodhostnik merged commit 3996ba8 into master Jul 24, 2026
2 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

Development

Successfully merging this pull request may close these issues.

Expose FlowFees fee-receiver addresses via a getter and a change event

2 participants