talk-android icon indicating copy to clipboard operation
talk-android copied to clipboard

Chat is not updated

Open mahibi opened this issue 1 year ago • 9 comments

Steps to reproduce

sometimes the chat screen doesn't refresh messages.

Expected behaviour

chat updated

Actual behaviour

chat not updated

mahibi avatar Jul 05 '23 17:07 mahibi

The issue happened another time after https://github.com/nextcloud/talk-android/pull/3147 was merged. So reopening as there must be more scenarios that trigger this bug.

mahibi avatar Jul 07 '23 09:07 mahibi

i guess somehow long-polling is interrupted because lookIntoFuture is false. Want to simplify the pullChatMessages logic in the long term..

mahibi avatar Jul 07 '23 09:07 mahibi

My question here; if an external signaling server is available, would it not make sense to make use of that to trigger pulling of new messages? Use long-polling as a fallback only, in case external signaling isn't available. Presumably, that could be done using the same connection that is used for the "{user} is typing..." messages.

ASerbinski avatar Jul 24 '23 14:07 ASerbinski

yes long polling should only be a fallback when external signaling server is available. Handling of many signaling messages still needs to be implemented (https://github.com/nextcloud/talk-android/issues/2816)

mahibi avatar Jul 24 '23 15:07 mahibi

I vote for this. Quite annoying, that new messages are not updated and chat has to be reopened.

foegra avatar Apr 06 '24 06:04 foegra

I vote for this. Quite annoying, that new messages are not updated and chat has to be reopened.

which app version are you using?

mahibi avatar Apr 17 '24 08:04 mahibi

I vote for this. Quite annoying, that new messages are not updated and chat has to be reopened.

which app version are you using?

I am running 19.0.0 RC4

foegra avatar Apr 22 '24 10:04 foegra

We are going to make huge changes for the chat in the next months (offline support). So we might not invest time to fix the "old" code but hopefully come up with a much better version that will fix it by the way..

mahibi avatar May 08 '24 13:05 mahibi

Sounds great!

foegra avatar May 08 '24 18:05 foegra