Skip to content

Mint/Transfer 1155 fix#696

Merged
sneakzttv merged 2 commits into
mainfrom
sneakz/transfer1155-fix
Nov 1, 2023
Merged

Mint/Transfer 1155 fix#696
sneakzttv merged 2 commits into
mainfrom
sneakz/transfer1155-fix

Conversation

@sneakzttv
Copy link
Copy Markdown
Contributor

@sneakzttv sneakzttv commented Nov 1, 2023

Problem:
Transfer 1155 wasn't working as we didn't have a mint function meaning we had an insufficient balance and it would throw an error.

Fix:
Created a test contract with an open mint function and added this to the unsorted samples class.
Added a button in the samples scene under more examples to showcase the working results.
Updated sample main scene and necessary scripts.
Closes #681

To test:

  1. Login and press the TransferErc1155 button
  2. Expected behavior: Transaction will throw an error due to insufficient balance.
  3. Press the Mint1155 button to mint an NFT to your account.
  4. Expected behavior: 1155 NFT mints to your account.
  5. Press the TransferErc1155 button.
  6. Expected behavior: 1155 NFT will transfer away from your account.

image

@sneakzttv sneakzttv closed this Nov 1, 2023
@sneakzttv sneakzttv reopened this Nov 1, 2023
Copy link
Copy Markdown
Contributor

@kantagara kantagara left a comment

Choose a reason for hiding this comment

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

LGTM

@sneakzttv sneakzttv merged commit a0571bd into main Nov 1, 2023
@sneakzttv sneakzttv deleted the sneakz/transfer1155-fix branch November 1, 2023 10:10
robGG1997 pushed a commit that referenced this pull request Nov 3, 2023
* Mint/Transfer 1155 fix

* Auto-duplicate Packages Samples

---------

Co-authored-by: sneakzttv <sneakzttv@users.noreply.github.com>
rob1997 pushed a commit that referenced this pull request Jan 16, 2025
* Mint/Transfer 1155 fix

* Auto-duplicate Packages Samples

---------

Co-authored-by: sneakzttv <sneakzttv@users.noreply.github.com>
sergeypanin1994 pushed a commit to sergeypanin1994/web3.unity that referenced this pull request Mar 16, 2025
* Mint/Transfer 1155 fix

* Auto-duplicate Packages Samples

---------

Co-authored-by: sneakzttv <sneakzttv@users.noreply.github.com>
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.

Transfer ERC 1155 example not working

3 participants