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

Initial Deploy Flow Sucks #236

Open
0xvangrim opened this issue Dec 12, 2022 · 5 comments
Open

Initial Deploy Flow Sucks #236

0xvangrim opened this issue Dec 12, 2022 · 5 comments
Assignees
Labels
Enhancement New feature or request

Comments

@0xvangrim
Copy link

Description:
When you deploy a Nimi page for the first time, we ask you to do two things:

First, you need to sign a message which contains your Nimi data.json contents.

Then, you need to sign a transaction to attach the IPNS key to your ENS content record.

Every time I make someone set it up, I tell them the first time they need to pay a txs fee to claim their profile. They then go to make one, get the request to do a signature and say “oh, actually I already got it gasless - that’s cool”. Most people then think it’s done and have claimed their nimi page before we even prompt the txs in their wallet.

Acceptance criteria:
For first time claiming, you don’t need to sign the txs - just setting the IPNS content is enough as verification as it proves the person owns the ENS domain.

@0xvangrim
Copy link
Author

@daveai @Mi-Lan See if the above is clear enough and if not you can add.

@0xvangrim 0xvangrim added the Enhancement New feature or request label Dec 12, 2022
@0xvangrim 0xvangrim assigned daveai and Mi-Lan and unassigned Mi-Lan and daveai Dec 12, 2022
@Mi-Lan
Copy link
Collaborator

Mi-Lan commented Dec 17, 2022

I was looking into this issue the main bottleneck is the api endpoint that generates IPNS hash /nimi/publish/ipns so this is more of an issue with the backend then the front-end below I provided screenshot of breakdown of 4.4second it takes
@adamazad can this be speeded up on the backend?
Screenshot 2022-12-17 at 16 58 58

@Mi-Lan Mi-Lan assigned adamazad and unassigned Mi-Lan Dec 17, 2022
@0xvangrim
Copy link
Author

0xvangrim commented Dec 19, 2022

@Mi-Lan, @adamazad - Look into if it is possible to only make the transaction and not the signature?

  • Create a new endpoint
  • Remove requirement of signature.

@adamazad
Copy link
Member

adamazad commented Jan 2, 2023

Blocked by dWeb, waiting on Toml

@adamazad
Copy link
Member

adamazad commented Jan 3, 2023

Toml from dWeb said this could be available by January 9th, 2022.

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants