-
Notifications
You must be signed in to change notification settings - Fork 34
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: authenticate as a guardian #389
Conversation
c069930
to
2f95f91
Compare
1a8fb29
to
9f56c9f
Compare
Should there be a warning to make sure one is in an environment where nobody can see/film their screen? |
@elsirion that's what I was doing with this, it shows that then you click reveal, do you want me to add more? |
97e7657
to
817e6db
Compare
password: password || '', | ||
}).toString(); | ||
|
||
return `guardian:authenticate?${params}`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should standardize the protocol prefix for all things Fedimint
817e6db
to
310a098
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some small nits but otherwise LGTM
bc4cda0
to
e8ceeb4
Compare
e8ceeb4
to
a5cae7b
Compare
Adds a button to toggle on
guardian:
in front of the invite code QR code so a client app can start an auth flow.