Notifications don't set a timestamp right now which means they implicitly have the same timestamp as when the notification is created.
The issue with that the timestamp from the notification is out of line with the timestamp of the message itself.
There's a potential disconnect in the timestamp of regular messages. In case of playbacks the timestamp will always be wrong the timestamp set on the message isn't used.
This especially important when the IRCv3 extension server-time is used.
Proposed change to fix this:
Set timestamp of notifications to timestamp of IrcMessage
Thaodan
commented
2 years ago
Notifications don't set a timestamp right now which means they implicitly have the same timestamp as when the notification is created. The issue with that the timestamp from the notification is out of line with the timestamp of the message itself. There's a potential disconnect in the timestamp of regular messages. In case of playbacks the timestamp will always be wrong the timestamp set on the message isn't used. This especially important when the IRCv3 extension server-time is used.
Proposed change to fix this: