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

Clearer validation and instruction on NWC strings #1710

Open
huumn opened this issue Dec 11, 2024 · 2 comments
Open

Clearer validation and instruction on NWC strings #1710

huumn opened this issue Dec 11, 2024 · 2 comments

Comments

@huumn
Copy link
Member

huumn commented Dec 11, 2024

Most people try to put the same string in for send/receive and our validation messages suck.

The rizful founder says everyone that tries to attach to SN is confused.

@ekzyis
Copy link
Member

ekzyis commented Dec 11, 2024

Interesting, they even have a good guide.

This is somewhat related to #1497.

I guess the message that both can't be the same sucks the most since it doesn't give you any information about what you should do instead.

@huumn
Copy link
Member Author

huumn commented Dec 11, 2024

Yeah, most people using NWC are only using one string and don't understand why we need two. Plus, other apps are storing send perms on their servers in plaintext. 😅

We probably need to go above an beyond explaining the reasoning and what permissions, exactly, each string requires.

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

No branches or pull requests

2 participants