42wim / matterircd

Connect to your mattermost or slack using your IRC-client of choice.
MIT License
294 stars 60 forks source link

Unable to reply to a multi-line message using any line's short thread ID #480

Closed kot0dama closed 1 year ago

kot0dama commented 2 years ago

When using short thread IDs, if someone sends a multi-line message, matterircd sends the messages to the client with auto-incremented short IDs for each line of the message.

Then when an IRC user wants to reply to the thread, it seems they need to use the last short thread-ID only, else matterircd will fail to detect which thread it belongs to, and will send @@xxx prefix in the message, without a warning it could not find the thread ID.

Also, there is absolutely no indication that a received line is part of a multi-line message, or that every line of these message belong to a specific thread ID, hence there is no way for a user to know they should reply using the last message's thread ID.

Example from IRC client side: matterircd_threadid_issues_hexchat_side

From Mattermost side: matterircd_threadid_issues_mm_side

Thank you

42wim commented 2 years ago

Try PR #482 this should fix the issue

kot0dama commented 2 years ago

Thanks for this, now we can clearly identify multi line messages.

Although, if I might nitpick, this change breaks 2 things:

Maybe we need to also prefix current thread ID and suffix re: / mention items when breaking the line before sending to IRC ?

hloeung commented 2 years ago

Follow up PR #483 to @42wim's PR #482 fixes the issue with having each line have the thread ID/context.

As for each line having the mention, as it is right now, it's only the last line. Would be a little noisy if it's highlighting on each and every message of a multi-line message, IMO.

kot0dama commented 2 years ago

Nice, thanks! I think having mentions on only one line is OK, it's just that we had them on every line of a multi-line message before the change.