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: clearer backup and restore info #750

Merged
merged 4 commits into from
Oct 24, 2024

Conversation

rolznz
Copy link
Contributor

@rolznz rolznz commented Oct 22, 2024

Fixes #645

Adds a link to our guide, makes copy on the backup and seed imports more clear and warns against importing from seed if you have open channels

image

image

@rolznz rolznz mentioned this pull request Oct 23, 2024
@rolznz
Copy link
Contributor Author

rolznz commented Oct 23, 2024

Addressed feedback:

  • More clear text in second checkbox on backup page, now also only shows when first checkbox is checked
  • Added checkbox on import page

@jpl-btc
Copy link

jpl-btc commented Oct 23, 2024

Alternative copy for the 3 warning sentences (feel free to use/remix/not use):

  • Your recovery phrase is a set of 12 words used to restore your savings balance from a backup.

  • Keep it safe and private to ensure your funds remain secure.

  • Please note, the recovery phrase cannot restore funds from lightning channels. If you had active channels on a different device, contact Alby support before proceeding. (note: the red color is not eye friendly and I have encountered colorblind users that had to select the text with the mouse to be able to read it, this happens also when setting up the unlock password)

@rolznz
Copy link
Contributor Author

rolznz commented Oct 24, 2024

@jpl-btc thanks for the feedback. Updated the copy and removed the red styling on the pages. The setup password second confirmation checkbox has bold text which differentiates it from the first confirmation box anyway, I think that's enough (and the destructive style was a bit buggy on the checkbox anyway because I couldn't find a way to change the fill color on the checkbox. CC @reneaaron )

@rolznz rolznz merged commit 70ac096 into master Oct 24, 2024
9 checks passed
@rolznz rolznz deleted the feat/clearer-backup-restore-info branch October 24, 2024 05:39
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.

Issue: Misunderstanding of Seed Phrase Restoration Scope
5 participants