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

Update MNS w/ enumerable extension #15

Open
7 tasks
Thykof opened this issue Jul 25, 2024 · 2 comments
Open
7 tasks

Update MNS w/ enumerable extension #15

Thykof opened this issue Jul 25, 2024 · 2 comments
Labels

Comments

@Thykof
Copy link
Contributor

Thykof commented Jul 25, 2024

Objective
Stop using the cache will greatly improve user experience, pain points:

  • after allocation a domain, user can't see it immediately
  • after change the owner of a domain, use still see it but can't do anything
  • after changing the target, use still see the previous target.

How to

  • implement NFT enumerable extension Implement ERC721 enumerable extension massa-standards#148
  • update the smart contract with the NFT enumerable extension
  • clean the smart contract code by removing all NFT functions: we must export them instead of copy-paste
  • upgrade the smart contract: call upgradeSC
  • update the frontend: don't use the cache of the explorer: the extension allows to get all NFT ids owned by the connected address. With that, we can use getDomainFromTokenId to get the list of owned domain and display them.
  • Migrate all registered / existing MNS in the new index
  • Test updated MNS system on labnet & buildnet
@SlnPons SlnPons added the p1 label Nov 21, 2024
@damip
Copy link
Member

damip commented Nov 21, 2024

  • create the enumerable extension
  • migrate the MNS SC to work with it (index the existing registered MNS names)
  • drop the need for the cache in:
    • the MNS front-end which allows it to become fully decentralized (including the deweb-hosted version)
    • the explorer / indexer

@SlnPons SlnPons changed the title Stop using the cache Update MNS w/ enumerable extension Nov 22, 2024
@SlnPons
Copy link
Collaborator

SlnPons commented Nov 22, 2024

This ticket is blocked by massalabs/massa-standards#148

@SlnPons SlnPons added the MNS label Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants