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

feat: SPACE (creation) WIZARDS 🧙 #69

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

travis
Copy link
Contributor

@travis travis commented Dec 15, 2023

First iteration on a space creation wizard for the console:

w3up console (1)
w3up console
w3up console (2)
w3up console (3)
w3up console (4)

This works, but I'm not totally happy with it - imho we could merge as-is but would love to iterate a bit more either before or after we do that.

Outstanding issues:

  1. There is currently no way to recover a space from a recovery phrase in the console. I think there's a way to do this in the CLI so we could potentially wait on this.
  2. "I'd prefer to back it up on your service" on the recovery phrase page doesn't do anything - imho it should run through the billing and recovery delegation automatically but I'm not convinced these are the right two options anyway.
  3. "Set up billing" seems weird, but that screen could potentially be a plan picker so it does feel like it needs to be there for now. In general I think the consensus in demos today was that we should move billing to the uploads page and just stop people from uploading until they have set up billing. This is complicated by the fact that we cannot set up space recovery until after billing has been set up. I think the right way to resolve this is to:
    a) Locally save the space (await ownedSpace.save()) generate and locally save the recovery UCAN (something like agent.addProof(await ownedSpace.createRecovery(account.did()))) during space creation
    b) When a user tries to upload, force them to go through the billing flow - plan selection if necessary, and then provisioning the space. Once that's done, offer to save the recovery delegation to the service.
  4. I'd like to take a pass at copy here with anyone interested - maybe could find some 8 or 9am PT time to do this before or after the holiday break

I'm inclined to fix all of these before merging, but that will probably end up being after the holiday break! Happy to find some shorter-term fixes for any issues and get this out if anyone would like it to ship urgently.

Based on the w3cli space creation flow, walk users through a process of setting their space up that allows them to save
the recovery phrase for the space and
recover from errors during the setup process.
also make loaders and className handling more consistent
Copy link
Contributor

github-actions bot commented Dec 15, 2023

Website preview 🔗✨

build log

@travis travis requested a review from gobengo December 15, 2023 21:56
@travis travis changed the title feat: space creation wizard feat: SPACE (creation) WIZARDS 🧙 Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant