Skip to content

Passkey (Seedless Restore)#89

Merged
dangeross merged 5 commits intomainfrom
savage-seedless-restore
Mar 10, 2026
Merged

Passkey (Seedless Restore)#89
dangeross merged 5 commits intomainfrom
savage-seedless-restore

Conversation

@dangeross
Copy link
Contributor

@dangeross dangeross commented Feb 6, 2026

Adds flows for creating or restoring using a Passkey with Seedless Restore

TODO:

  • Add Breez NOSTR relay

@vercel
Copy link

vercel bot commented Feb 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
breez-glow-web Ready Ready Preview, Comment Mar 10, 2026 1:50pm
breez-sdk-spark-example Ready Ready Preview, Comment Mar 10, 2026 1:50pm
savage-glow-web Ready Ready Preview, Comment Mar 10, 2026 1:50pm

@dangeross

This comment was marked as outdated.

@dangeross
Copy link
Contributor Author

dangeross commented Feb 6, 2026

  • Let's give a default value for the name (Glow) if possible
  • Avoid technical terms (remove FIDO)

@dangeross

This comment was marked as outdated.

@dangeross

This comment was marked as outdated.

@kingonly
Copy link
Member

@dangeross

  • Let's give a default value for the name (Glow) if possible
  • Avoid technical terms (remove FIDO)
  • "Restore with phrase" instead of "Use recovery phrase instead" and "Restore with recovery phrase"
  • Why are we using "account"/"accounts"? Is this the right term to use? Why do we have "create new account" in the restore page?

@dangeross
Copy link
Contributor Author

  • Why are we using "account"/"accounts"? Is this the right term to use? Why do we have "create new account" in the restore page?

@kingonly I was struggling to find the correct, non techy noun, I thought maybe Account/Profile/Identity.

"Create New Account" can be removed now we have the Other - manual entry option

@dangeross dangeross changed the title Seedless restore Seedless restore (Passkey) Feb 27, 2026
@dangeross dangeross force-pushed the savage-seedless-restore branch from e4f7d70 to 6119ebd Compare February 27, 2026 09:55
Copy link
Contributor

@erdemyerebasmaz erdemyerebasmaz left a comment

Choose a reason for hiding this comment

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

Looks solid. Left some question about some edge cases, the rest are structural & clean code suggestions.

Copy link
Member

@roeierez roeierez left a comment

Choose a reason for hiding this comment

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

Except for Erdem review LGTM.

@dangeross
Copy link
Contributor Author

Added a feature gate for Passkey - clicking the Glow logo 5 times will activate it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants