Skip to content

chore: more friendly error message#329

Merged
osa1 merged 1 commit into
osa1:masterfrom
ttys3:more-friendly-errmsg
May 25, 2021
Merged

chore: more friendly error message#329
osa1 merged 1 commit into
osa1:masterfrom
ttys3:more-friendly-errmsg

Conversation

@ttys3

@ttys3 ttys3 commented May 25, 2021

Copy link
Copy Markdown
Contributor

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.

Comment thread crates/tiny/src/ui.rs Outdated
@ttys3
ttys3 force-pushed the more-friendly-errmsg branch from 37d8665 to ed61401 Compare May 25, 2021 18:15
@osa1

osa1 commented May 25, 2021

Copy link
Copy Markdown
Owner

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