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

web: rewrite and update prax wallet section #55

Closed
wants to merge 1 commit into from

Conversation

redshiftzero
Copy link
Collaborator

This PR rewrites and updates the first page of the "Using Penumbra on the web" section.

There are two flows here:

  • using quests
  • not using quests

I've gone through both flows, and adjusted things a bit to reduce duplication (e.g. some of the content got displayed again through quests). I modified the wallet install quest to break it up into two chunks - one for the install/connect, and one to generate the addresses after generating a seed phrase.

The RPC endpoint selection and frontend selection was outdated, so I've rewritten these sections.

Terminology changes:
"Recovery phrase", "seed phrase" -> "recovery seed phrase"
"passphrase" -> "password" (to be consistent with what Prax is using)

@TalDerei TalDerei mentioned this pull request Feb 17, 2025
@TalDerei TalDerei self-requested a review February 18, 2025 02:24
Copy link

@TalDerei TalDerei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these changes are mutually exclusive with #42 (not sure about its current status? @hdevalence). Offline on discord, I shared some relevant onboarding guide specs and designs from RC, and I'd coordinate with them to that end towards a holistic onboarding redesign.

@redshiftzero
Copy link
Collaborator Author

If #42 is ready, happy to rework this after that is merged

@redshiftzero
Copy link
Collaborator Author

Closing this for now

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.

2 participants