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

LNDHub: Custodial Warning: add option to dismiss #2344

Closed
kaloudis opened this issue Aug 6, 2024 · 2 comments
Closed

LNDHub: Custodial Warning: add option to dismiss #2344

kaloudis opened this issue Aug 6, 2024 · 2 comments
Assignees
Labels
LNDHub Issues specific to LNDHub
Milestone

Comments

@kaloudis
Copy link
Contributor

kaloudis commented Aug 6, 2024

Many users have been complaining about the LNDHub warning we've added to the wallet header in the app.

On the warning page, let's give the users the option to dismiss the warning. In order to stifle bad actors, let's make the users very explicitly opt out with a list of checkboxes to make sure they understand the risks in dismissing the warning.

@kaloudis kaloudis added the LNDHub Issues specific to LNDHub label Aug 6, 2024
@kaloudis kaloudis added this to the v0.9.1 milestone Aug 6, 2024
@newtonick
Copy link
Contributor

+1 on this option.

I would like the option to disable the LNDHub custodial warning for LNDHub wallet/accounts I personally have custody of. I'm willing to jump through many hoops to make it harder for bad actors to take advantage of zeus users.

@kaloudis
Copy link
Contributor Author

First draft of the checkbox text:

  • I know that I don't have custody of the funds in this wallet without the 12 or 24 word seed phrase
  • I know that an LNDHub account password is not the same as a seed phrase
  • I either set up this connection myself or I trust the person who set it up
  • I'm not just lying to get this over with

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LNDHub Issues specific to LNDHub
Projects
None yet
Development

No branches or pull requests

3 participants