-
Notifications
You must be signed in to change notification settings - Fork 212
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
NFT Vault app smart contracts #44
Comments
You had me at cryptokitties 💻🐱 |
This should probably also be reflected in the finance app somehow, or maybe we need a separate app for NFTs. What do you think would make more sense? Given NFTs will rarely be used in a budget, it probably makes more sense to have a dedicated NFT app to list your NFTs. Thankfully, ERC721 specifies names, descriptions and images for the different NFTs, so it would probably be trivial to list them at least. |
I totally agree. I highly doubt NFTs will be used to manage anything financial, in the way the current finance app works (accounting for payments in and out). Maybe something like "Assets" that sits alongside Tokens? |
I like the name "Assets" except assets could also cover cash reserves, so it might be confusing. |
ERC721 was just merged. We need an ERC721 vault :) ethereum/EIPs#841 |
Current balance: 0.000000 ETH |
Hi!, a very interesting problem, I would give it a try, if none at the core team is able to do it. The idea would be to borrow the ERC721 standard implementation from our zeppelin cousins, and build the NTF Vault (Inventory app?) from there, adding the basic functionality. |
* WIP add token to DAO * WIP bump aragon-cli version and add tokens to DAO * Seed employees and tokens to DAO * Bump aragon cli version * Add allowed tokens * Use entity domain as employee name * Fix Payroll tests after merge master * Fix Payroll tests, change role name * Fix more tests * Start with 9 accounts and add accounts to entities mock
ERC721 defines the standard for Non-Fungible Tokens (NFTs). By supporting ERC721 Aragon organizations will be able to manage NFTs.
Decentraland's LAND and Cryptokitties both use ERC721 NFTs and seems like it will become the standard for future tokens.
Check the Token Vault for reference and inspiration
The text was updated successfully, but these errors were encountered: