Skip to content
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

Distribute NFT child tokens with burning the qty=1 parent ones #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zh
Copy link
Contributor

@zh zh commented Dec 26, 2020

The code from PR #2 require too many changes. The branch will be closed and the current one will replace it.

The goals for this PR:

  • As little changes as possible to the original code:
    • server.ts - just one function change to point to a wrapper function generating NFTs and usual tokens
    • slpfaucet.ts - a small wrapper function - tokenSend()
  • No NFT logic in server.ts - should be unaware what kind of tokens are distributed
  • All NFT logic extracted in a separate file - faucet_utils.ts contains all NFT related code (in fact just one function)

The faucet is working as it is now. As a future performance optimization (in a separate PR maybe) will try to combine creating single quantity parent token UTXO and NFT child token genesis in one call to the blockchain.

@zh zh mentioned this pull request Dec 26, 2020
@blockparty-sh
Copy link
Member

@jcramer I have this running and seems to work here fine + looked this over. Wanna take a second look at it?

@jcramer
Copy link
Contributor

jcramer commented Jan 14, 2021

@blockparty-sh Does this PR have any updates you've found necessary for the waifufaucet? I don't have time to test this out so I'll let you merge it when its ready.

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