'All Messages (loud)' setting considers room avatar change and leaves stuck notification

[turt2live]

Description

While setting up a new room (with no messages) I intentionally set the notification level to 'All Messages (loud)'. Another user changed the room avatar and that caused an audible and visual notification that now can't be removed.

Unsure if this applies to any other case besides this specific case, so the reproduction steps are for this known case. Possibly related to #3363.

Edit: Sending a message after the notification will allow you to clear the notification

Steps to reproduce

• Create a new room and invite someone to it
• Set your notification setting to 'All Messages (Loud)'
• Be active in another room while the someone you invited sets the room avatar
• Note audible notification and visual indicator on room

I would have expected silence. The room doesn't record a 'room avatar changed' event and it's unclear as to why I got the notification. Not to mention I can't dismiss the notification.

Log: not sent (unable)

Hopefully helpful but probably just excess filler screenshots:


(avatar changed after invites)

Version information

• Platform: web (in-browser)
• Browser: Chrome 56
• OS: Windows 10
• URL: riot.im/develop

[lukebarnard1]

WRT the notif not disappearing: I bet this has something to with the membership EventListSummary (MELS) at the bottom. Maybe a read receipt can't be sent for the most recent event because it's been hidden by the MELs.