-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Update std-lib to reflect new granular asset minting spec #4651
Conversation
I think worths talking discussing if we should have two libraries in the std lib, one with the same api as before (single token), and one with the above multi-token api. Many usecases only really need one token, and thus would make the new api overtly verbose |
+1 on @SwayStar123's comment Would be nice to keep |
### Abstract This PR introduces granular asset minting which would enable sub identifiers to be included at asset creation. These sub identifiers can include important metadata and be used to enable native level NFTs. **This adds:** - A new asset ID construction (which incorporates the contract ID and the sub asset ID). - New receipt logs for minting and burning which are helpful for indexing. - Changes to the MINT and BURN opcodes. ### Cons - This removes the nice feature that asset IDs are the contract ID. ### Pros - Native level UTXO based NFTs. - Single contract multi-asset pools (useful for common AMM designs). Implementation tickets/PRs: - FuelLabs/fuel-vm#496 - FuelLabs/fuels-ts#1067 - FuelLabs/fuel-indexer#1012 - FuelLabs/sway#4651 --------- Co-authored-by: Brandon Kite <brandonkite92@gmail.com> Co-authored-by: Green Baneling <XgreenX9999@gmail.com>
Closed by #4821 |
Description
Changes to std-lib to support multi-asset native token contracts with the introduction of a sub id.
Closes #4644
Checklist
Breaking*
orNew Feature
labels where relevant.