Eidenz / Nextcloud-Talk-Desktop

A small desktop app for Nextcloud Talk notifications using the Spreed API.
7 stars 2 forks source link

Talk not showing new messages without site refresh #5

Closed LucaOverflow closed 2 years ago

LucaOverflow commented 2 years ago

Hi again,

after using you app for a bit, I noticed Talk is not showing new messages if you dont refresh the site. This is a pretty deep issue in Talk itself I guess. But since they probably wont do anything to fix it, maybe automatically refreshing the site when a new message gets deteced would be a workaround.

I didn't dove in the API or your application yet, but maybe there is even a more elegant way to do this.

As an example for this problem: Let's assume that I'm in Talk chatting with someone. At the same time there is a new message from someone else. Since the Talk window is focused it doesn't trigger a notification (which is good, at least when it would be a message from the chat I'm in right now). But since the view of talk does not refresh I dont know that I got a new unread message. Only after I refresh the page I could see that.

I guess there are some problems with hard refreshing, but I don't have a better idea. Reloading in a Meeting would probably pretty stupid. Also reloading while typing would be pretty annoying. So the fix would have to decide smart on when to reload. Maybe just after ending a call or sending a message.

If the App could see which chat is selected it could just hide notifications from that one. The refreshing would still be needed to work optimally. But this would allow new messages to be recieved even while in a call.

Maybe someone has a better idea how this could be done. However @Eidenz thank you very much for the work you put into this app.

LucaOverflow commented 2 years ago

Nevermind. I just had the situation again and it worked. Weird...