-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
TuxedoDapp #2112
TuxedoDapp #2112
Conversation
1st iteration of the proposal
Removed the smart contract part and piece_contract development , since it is not the scope of Dapp development . Smart contract development needs support from underlaying tuxedo runtime.
cleaning up
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read and hereby sign the Contributor License Agreement. |
1 similar comment
I have read and hereby sign the Contributor License Agreement. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AltiMario could you add the file extension .md
?
Some further thoughts:
- for UI-based deliverables (such as the wallet or the dApp) we'd require you to show us some wireframes, so we know what to expect
- M2 is currently very vague. Please specify which features you're planning to implement there (e.g. "manage their NFT-based pet collections" => what does "manage" include exactly?)
For the sake of completeness I'm going to link the two mentioned issues here:
- Full Tuxedo App Implementation Off-Narrative-Labs/Tuxedo#76
- Wallet: Support transactions to multiple recipients Off-Narrative-Labs/Tuxedo#62
(cc @JoshOrndorff, would you be open to assist the MLabs team in case of questions during the implementation if we end up signing a grant?)
Finally, @AltiMario the price is quite a bit higher than initially anticipated. I was wondering if you'd be open to reduce the price.
Updated the file extension to .md
Updated below : 1. Meaning of Feature Parity 2. Updated browsers which will be supported . 3. Meaning and Scope of "Manage NFT" in M2(Dapp) section
Updated the semantic issues.
Updated the M2 features i.e DApp features supported to Kitties based on the features by Kitties pieces in blockchain code .
updated M2 deliverables in all places in accordance with kitties pieces.
I'm excited to see this proposal! And I'm certainly willing to answer questions and provide guidance. I want to call your attention to Off-Narrative-Labs/Tuxedo#30 as well. The current CLI wallet is developed by using the same types as the template runtime. If a chain developer comes along and adds or removes tuxedo pieces, small corresponding changes need to be made to the wallet. You may choose to design your wallet in this way as well, and I think such a wallet is very valuable. A much more elegant and general solution would be to have a standard metadata format like FRAME has (although it can be much simpler than FRAME's especially at first). This would allow wallets to detect the existence of a particular Tuxedo piece and adapt accordingly like Polakdot JS Apps does. Finally, regarding the price, I am not w3f or grants committee, so I'll just share these few thoughts because. Our original Tuxedo Grant was to develop Tuxedo Core, Template, and CLI wallet. If the scope is just to achieve feature parity with the CLI wallet plus fix #62, I'd say that is a smaller effort than our original grant. If the scope includes a metadata format and dynamic wallet, I expect it to be a similar or perhaps slightly larger effort than the original. |
Updated DApp functionalities to support the trading, updating features, searching of kitties ,etc .
Updated the IPFS requirement to store meta data of kitty.
Wireframes links are updated.
Fixed typos
After a better analysis of the scope, we decided to cut the cost by 10% but maintain 3 months as the total estimated duration.
fixed some typos and comments.
KYC/KYB checks completed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the application and the updates, @AltiMario. Happy to approve.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AltiMario thanks for the updates, LGTM now 👍
db38448
This reverts commit db38448.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re-approving this.
@keeganquigley @semuelle feel free to re-approve, @AltiMario accidentally overwrote and reverted the changes, hence there are no changes since the approval.
So there are already three approvals right? Is anything blocking this? |
KYB still needs to be completed. |
@AltiMario seems to think that is completed. See #2112 (comment) |
Thanks @JoshOrndorff looks like the latest submission is still pending, we should have results on this tomorrow. |
I apologize for the inconvenience, but there seems to be an error with the document that the MLabs administration previously signed. I believe due to the recent holiday, they forgot to address this issue. |
@AltiMario, who should I reach out to and how to discuss the KYB? |
hi @semuelle the person in charge of the KYB is Deepak Kumar ([email protected]) |
Congratulations and welcome to the Web3 Foundation Grants Program! Please refer to our Milestone Delivery repository for instructions on how to submit milestones and invoices, our FAQ for frequently asked questions and the support section of our README for more ways to find answers to your questions. |
Awesome, thanks for the opportunity and for supporting me in the process! |
Sorry it took this long, @AltiMario. We finally managed to resolve the KYC/KYB issues last night. |
Project Abstract
Develop a Tuxedo Web Wallet and Dapp.
Our goal is to showcase the potential of Polkadot using the UTXO paradigm for the end user, leveraging Tuxedo. The project will consist of two key stages:
Grant level
Application Checklist
project_name.md
).@_______:matrix.org
(change the homeserver if you use a different one)