Skip to content

fix(sdk-coin-eth): add enable token support for zama#8906

Merged
MohammedRyaan786 merged 1 commit into
masterfrom
CHALO-472
Jun 1, 2026
Merged

fix(sdk-coin-eth): add enable token support for zama#8906
MohammedRyaan786 merged 1 commit into
masterfrom
CHALO-472

Conversation

@MohammedRyaan786
Copy link
Copy Markdown
Contributor

@MohammedRyaan786 MohammedRyaan786 commented Jun 1, 2026

TICKET: CHALO-472

This pull request primarily updates the handling of token enablement transactions for Ethereum-like coins. The main changes include adding support for a new transaction type (enabletoken), centralizing token enablement configuration in the base Eth class, and updating transaction intent handling to support token enablement.

Token Enablement Support:

  • Added 'enabletoken' to the list of supported transaction types in AbstractEthLikeNewCoins, allowing the system to recognize and process token enablement transactions.
  • Moved the getTokenEnablementConfig implementation from Erc7984Token to the base Eth class, ensuring consistent token enablement configuration across all Ethereum-like coins. [1] [2] [3] [4]

Transaction Intent Handling:

  • Updated MpcUtils to handle the new 'enableToken' intent type, supporting the necessary parameters for token enablement transactions.

@linear-code
Copy link
Copy Markdown

linear-code Bot commented Jun 1, 2026

CHALO-472

@MohammedRyaan786 MohammedRyaan786 marked this pull request as ready for review June 1, 2026 14:38
@MohammedRyaan786 MohammedRyaan786 requested review from a team as code owners June 1, 2026 14:39
Copy link
Copy Markdown
Contributor

@bhuvanr159 bhuvanr159 left a comment

Choose a reason for hiding this comment

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

lgtm

@MohammedRyaan786 MohammedRyaan786 merged commit 129d44e into master Jun 1, 2026
22 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.

3 participants