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

chore: more friendly error message #329

Merged
merged 1 commit into from
May 25, 2021
Merged

Conversation

ttys3
Copy link
Contributor

@ttys3 ttys3 commented May 25, 2021

It confused me for a night when I try to send /msg NickServ REGISTER passwd email and the tui always gives me "use /connect <server> to connect to a server" error.

But actually I have an connected server. so I got confused.

But if I send /msg irc.libera.chat NickServ REGISTER passwd email, it succeed!

After read the code, I found that this is not the right way to send a msg to NickServ (thought it works).

The only thing I need to do is switch from the default mentions tab to next tab.

and then, /msg NickServ REGISTER passwd email works as expected.

@ttys3 ttys3 force-pushed the more-friendly-errmsg branch from 37d8665 to ed61401 Compare May 25, 2021 18:15
@osa1
Copy link
Owner

osa1 commented May 25, 2021

Thanks @ttys3 for the patch and thanks @trevarj for the review :-)

@osa1 osa1 merged commit 9d2d212 into osa1:master May 25, 2021
@osa1 osa1 added changelog and removed changelog labels May 25, 2021
osa1 added a commit that referenced this pull request May 25, 2021
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.

3 participants