-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEAT] ERC-1155 Support #3301
[FEAT] ERC-1155 Support #3301
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments and questions.
package.json
Outdated
@@ -87,7 +87,7 @@ | |||
"dependencies": { | |||
"@exodus/react-native-payments": "https://github.com/wachunei/react-native-payments.git#package-json-hack", | |||
"@metamask/contract-metadata": "^1.30.0", | |||
"@metamask/controllers": "^17.0.0", | |||
"@metamask/controllers": "./metamask-controllers-v17.0.0-erc-1155-v2.tgz", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update path once controller is released
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the controllers dependency and it looks good.
…obile into feat/erc1155
…into feat/erc1155
…obile into feat/erc1155
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌮 🌮 🌮 🌮 🌮 🌮
…obile into feat/erc1155
Description
Currently we don't support the collectible standard ERC-1155, the focus of this development is to integrated the ERC-1155 ABI to access the appropriate methods of the smart contract.
To Do