Skip to content

feat: add zama to coinFactory#8739

Merged
MohammedRyaan786 merged 1 commit into
masterfrom
CHALO-402
May 12, 2026
Merged

feat: add zama to coinFactory#8739
MohammedRyaan786 merged 1 commit into
masterfrom
CHALO-402

Conversation

@MohammedRyaan786
Copy link
Copy Markdown
Contributor

@MohammedRyaan786 MohammedRyaan786 commented May 12, 2026

TICKET: CHALO-402

This pull request adds support for ERC-7984 confidential tokens to the BitGo platform, including their registration, instantiation, and handling throughout the codebase. The changes introduce a new Erc7984Token class, update token registration logic, and ensure proper test coverage and exports.

ERC-7984 Token Support

  • Added a new Erc7984Token class in sdk-coin-eth, implementing confidential token logic and configuration, including constructor, metadata accessors, and transaction rules.
  • Updated imports and exports across sdk-coin-eth and BitGo modules to include Erc7984Token, Erc7984TokenConfig, and Erc7984Coin. [1] [2] [3] [4] [5]

Token Registration and Factory Integration

  • Modified the coin factory and registration logic to handle ERC-7984 tokens, including dynamic constructor creation and proper registration in both main and test networks. [1] [2] [3] [4]
  • Updated the getTokenConstructor function to instantiate Erc7984Token when appropriate, based on token type and configuration.

Testing and Validation

  • Expanded unit tests to verify ERC-7984 token registration, constructor counts, and global coin map integration, ensuring parity with ERC20 and ERC721 token handling. [1] [2] [3]
  • Added test coverage for ERC-7984 token presence in browser tests.

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 12, 2026

CHALO-402

@MohammedRyaan786 MohammedRyaan786 marked this pull request as ready for review May 12, 2026 07:13
@MohammedRyaan786 MohammedRyaan786 requested review from a team as code owners May 12, 2026 07:13
@MohammedRyaan786 MohammedRyaan786 merged commit 9001be2 into master May 12, 2026
33 of 34 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.

2 participants