Skip to content
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

Unread mention notification does not go away if the message in question has been deleted #13286

Open
max-nextcloud opened this issue Sep 12, 2024 · 1 comment · May be fixed by #13355
Open
Labels
1. to develop bug feature: api 🛠️ OCS API for conversations, chats and participants feature: chat 💬 Chat and system messages

Comments

@max-nextcloud
Copy link

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Login as one user, send a direct message to another user.
  2. Delete the message.
  3. Login as the other user, see the unread mention button
  4. Click the button - conversation scrolls down but no unread messages visible.
  5. Reload the page - unread mention button still there.

Expected behaviour

Either the unread mention button should not show
or it should disappear after the scrolling and opening the conversation with the deleted message for the first time.

Actual behaviour

The unread mention only went away after staying in the conversation in question for a prolonged time (to take a screenshot).

Talk app

on cloud.nextcloud.com

Talk app version: 20.0.0-rc4

Browser

Firefox

Browser log

grafik
grafik

Server configuration

cloud.nextcloud.com

@nickvergessen nickvergessen added 1. to develop feature: chat 💬 Chat and system messages feature: api 🛠️ OCS API for conversations, chats and participants and removed 0. Needs triage labels Sep 12, 2024
@nickvergessen nickvergessen added this to the 💚 Next Patch (29) milestone Sep 12, 2024
@Souptik2001 Souptik2001 linked a pull request Sep 19, 2024 that will close this issue
14 tasks
@Souptik2001
Copy link
Contributor

@nickvergessen Seems like we just had to bust the unread cache on the delete message action. I checked to see if there will be any regression issues due to this seems like no.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop bug feature: api 🛠️ OCS API for conversations, chats and participants feature: chat 💬 Chat and system messages
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants