Skip to content

DRAFT: feat(transactions): detect burn address in ICRC token send flow#7831

Draft
yhabib wants to merge 3 commits intomainfrom
feat/burning-sns-tokens
Draft

DRAFT: feat(transactions): detect burn address in ICRC token send flow#7831
yhabib wants to merge 3 commits intomainfrom
feat/burning-sns-tokens

Conversation

@yhabib
Copy link
Copy Markdown
Contributor

@yhabib yhabib commented Apr 20, 2026

Motivation

When a user sends ICRC/SNS tokens to the minting account (which is the burn address in ICRC-1), the transaction incurs no fee. It should be clear to the user that they are burning tokens, not just sending them to a regular address.

Changes

  • Added the queryIcrcMintingAccount API function to fetch the minting account from an ICRC-1 ledger canister.
  • Retrieved the minting account on mount in IcrcTokenTransactionModal and passed it down as burnAddress.
  • Added the burnAddress prop to TransactionModal and TransactionForm to propagate the burn address through the send flow.
  • Displayed a "Burn address" label below the destination input when the entered address matches the minting account.
  • Hidden the transaction fee and used 0n as the effective fee when the destination is the burn address, both for the maximum amount calculation and the actual transfer.

Tests

Todos

  • Accessibility (a11y) – any impact?
  • Changelog – is it needed?

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.

1 participant