-
-
Notifications
You must be signed in to change notification settings - Fork 442
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
Add better support for IRC private messages #4158
Conversation
clang-tidy review says "All clean, LGTM! 👍" |
clang-tidy review says "All clean, LGTM! 👍" |
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.
clang-tidy made some suggestions
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
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.
Looks good to me, feel free to merge this in!
Pull request checklist:
CHANGELOG.md
was updated, if applicableDescription
They now look more like twitch whispers with it showing the actual nick colored like in other channels. I've included support for functionality actually added #4157, but this PR doesn't depend on that one or the other way around. With that PR replayed private messages (for example by ZNC) will work.
TODO:
I've decided to implement this in another PR since there would be a need to make builtin commands work on IRC.Back on the menu since Make builtin commands work in IRC channels #4160 was merged.