Marcel Müller
Marcel Müller
> [...] (and reactions?). Yep. These are the messages we currently consider an "update message": https://github.com/nextcloud/talk-ios/blob/c53dd7b6cb05d793a542d2ca3ed69c5ac98e2933/NextcloudTalk/NCChatMessage.m#L254-L262 When we receive such a message, we update the corresponding message and the quotes...
> Is it reactions you are seeing (like me) or any notifications? The test above was with a normal chat message
I think it's due to the change in https://github.com/nextcloud/spreed/pull/10544, where we set `markNotificationsAsRead` always to `0`: https://github.com/nextcloud/spreed/blob/06af15480324de78c7f4e5365e2164a9921794b6/src/services/messagesService.js#L72-L72 To improve the situation I think it makes sense to set the parameter...
But still would remove existing notifications for that conversation as soon as we receive a message ?
> But not when the user is no longer looking at it Right... So should we add a parameter to the mark as read endpoint so the web could remove...
Not sure about that: https://github.com/nextcloud/spreed/actions/runs/7739390969/job/21102764394#step:14:1184 Is this known?
> But it should never fail to poll? Where is this coming from all a sudden? I only see it since yesterday... Oh, is this new? Then let me investigate......
Doesn't seem to be an issue in recent runs. So would close this for now.
> * Amend to the author name ` (edited)` / ` (edited by moderator-name)` This could be tricky on phones, I can see that most of the time the moderator...
> Agree, we might need to find a better place for mobile devices. Maybe we could add a pencil to the message to indicate it was edited and show a...