Skip to content

fix: add bridged HRC721 mint and burn API - #86

Open
blueforyou0202 wants to merge 1 commit into
blockcoders:mainfrom
blueforyou0202:fix-bridged-hrc721-mint-burn-api
Open

fix: add bridged HRC721 mint and burn API#86
blueforyou0202 wants to merge 1 commit into
blockcoders:mainfrom
blueforyou0202:fix-bridged-hrc721-mint-burn-api

Conversation

@blueforyou0202

Copy link
Copy Markdown

Summary

  • add BridgedHRC721Token.mint(account, tokenId, txOptions) for the ABI's mint(address,uint256) function
  • add burn(tokenId, txOptions) and burnFrom(owner, tokenId, txOptions) for the bridged HRC721 burn functions
  • add focused unit coverage for the new SDK wrapper dispatches

Notes

This is a separate bridged HRC721 mint/burn API gap. It intentionally does not touch tokenURI, ownership/approval APIs, safe-transfer APIs, or overloaded safeMint handling.

Validation

  • Docker node:16-bullseye with only this repo mounted
  • npm ci --ignore-scripts --no-audit --fund=false
  • ./node_modules/.bin/mocha -t 120000 --require ts-node/register src/tests/bridged-hrc721-token-mint-burn.spec.ts --exit with --network none
  • ./node_modules/.bin/tsc --noEmit --pretty false -p tsconfig.json with --network none
  • ./node_modules/.bin/eslint src/bridge/bridgedHrc721Token/index.ts src/tests/bridged-hrc721-token-mint-burn.spec.ts with --network none

Related: harmony-one/bounties#125

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