Skip to content
This repository was archived by the owner on Dec 16, 2022. It is now read-only.

Make "msg" packet support multiple messages and remove "more" packet #594

Closed
xPaw opened this issue Jan 18, 2016 · 3 comments
Closed

Make "msg" packet support multiple messages and remove "more" packet #594

xPaw opened this issue Jan 18, 2016 · 3 comments

Comments

@xPaw
Copy link
Contributor

xPaw commented Jan 18, 2016

Doing so will mean messages loaded from history are processed exactly as newly incoming messages. (links/channels are parsed correctly, etc.). But to support this, "msg" message needs to be able to accept multiple messages, and then know how to insert them into DOM based on time the message was sent.

@AlMcKinlay
Copy link
Contributor

I think that's a sensible way to deal with messages anyway, as there could be a race condition with messages the way they currently work.

@xPaw
Copy link
Contributor Author

xPaw commented Jan 18, 2016

This would go nicely with server-time extension: http://ircv3.net/specs/extensions/server-time-3.2.html

@astorije
Copy link
Collaborator

@xPaw, doing so will also make #588 (and others) better when reloading or loading logs, correct?
Do you want to give a try at this?

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

No branches or pull requests

3 participants