-
Notifications
You must be signed in to change notification settings - Fork 3
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
Pending status for outgoing XMPP messages #244
Conversation
app/controllers/base_channel.js
Outdated
}); | ||
|
||
if (this.model.protocol === 'XMPP') { |
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.
Why only for XMPP? Wouldn't it be the same process for IRC?
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.
Nope. You don't receive your own messages back from a channel in IRC.
Also fixes own messages being missing in channel history.
471fc2e
to
cc00d2e
Compare
The dependency label is almost always orange, so let's change the sort order to make this look nicer.
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.
Nice one 👍
This marks outgoing messages as pending, and then confirms that they were sent when received back from the room. It also fixes the problem of one's own messages missing in the chat history that is sent by XMPP servers upon joining a channel.
Note: will probably fail with DMs (#242), because we would need message carbons for those. Hence the WIP, until DMs are merged.Fixed for DMs.P.S.: I think we'll want to have a timeout for showing messages as pending in the UI later on. Currently, there's a bit of a flash from switching the text color after a split second. And we can also add an actual text marker or other indicator, after a few seconds of it not having confirmed.