Some rooms do not get reloaded when `messagesNeedsReload` is fired

prose-im / prose-app-web

Prose Web application. XMPP client for team messaging.

https://prose.org/downloads

Mozilla Public License 2.0

25 stars 2 forks source link

Some rooms do not get reloaded when `messagesNeedsReload` is fired #109

Closed valeriansaliou closed 5 months ago

valeriansaliou commented 5 months ago

From @nesium :

To reproduce:

Select conversation A in account X
Go offline/turn off Wifi
Send message to conversation A from account Y
Go online again with account X
messagesNeedsReload should be called for conversation A

At that point I would expect the web app to call loadLatestMessages and display the loaded messages so that those sent from account Y are visible.

valeriansaliou commented 5 months ago

Ok, so I've tried what you said @nesium ; but I never receive a delegate event messagesNeedReload from the core when reconnecting account X, which should receive a new message from account Y (sent while account X was offline).

I tried replicating this scenario for both a one-to-one chat and a MUC channel. Same behavior.

I believe the issue you described here comes from the core, not the app. Could you confirm?

valeriansaliou commented 5 months ago

I believe I found out the reason, which was unrelated. I could reproduce a similar issue, which I believe is the one you've seen as well. Fixed in 2a2d8dc2082d63ba4d98367810b7f3040fc11fd8