You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
claim should be renamed to eliminate confusion on what it does for the person running it (maybe create or process)
mint-many should be converted into a single mint function instead of iterating over a list
Thinking some more on this - maybe claim and mint should be combined into one public function? It would accept the BTC TX, and as long as it's valid per the requirements, complete the mint. Will definitely tie into #18.
The text was updated successfully, but these errors were encountered:
Looking over the default template, there are a few things:
MAX-SUPPLY
and referencesWALLET
can be removed in favor of BTC receiver address (Add swap parameters to NFT contract #21)ERR-SOLD-OUT
may not be necessaryclaim
should be renamed to eliminate confusion on what it does for the person running it (maybe create or process)mint-many
should be converted into a single mint function instead of iterating over a listThinking some more on this - maybe
claim
andmint
should be combined into one public function? It would accept the BTC TX, and as long as it's valid per the requirements, complete the mint. Will definitely tie into #18.The text was updated successfully, but these errors were encountered: